-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes for the pack()
and unpack()
method
#1401
Conversation
@jyang2009 Can you provide a bit more details about the commands you were executing that resulted in this error? |
Hi Jan, I just used the code that I used to copy jobs to the new project, which was working previously. |
Hi Jing, I am sorry for the delay. The error message basically says, that
My assumption would be it is just a
This should resolve this issue. Best, Jan |
for more information, see https://pre-commit.ci
@samwaseda Originally I was waiting for feedback from @jyang2009 if this solves her problem, as from my side this looks ready to be merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would have probably changed the error message and kept the current functionality, but I'm ok with the suggested solution.
Side note: I don't think the csv file should be separated from the tar file, so I will probably remove it entirely in this PR |
@jyang2009 based on our discussion I created a couple of fixes for the
pack()
andunpack()
method. Can you check if these resolve the issues you had with archiving onptmp
?