Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Question: anonynmous file #456

Closed
alethomas opened this issue Jan 26, 2022 · 3 comments · Fixed by #457
Closed

Question: anonynmous file #456

alethomas opened this issue Jan 26, 2022 · 3 comments · Fixed by #457
Assignees
@thomasbtf @simakro

Comments

@alethomas
Copy link
Collaborator

uncovar/workflow/rules/long_read.smk

Line 55 in e844fc4

"results/.indicators/replacement_notice.txt",

can this anonynmous addressing of the file cause problems, when were removing it with the next rule?
@alethomas
Copy link
Collaborator Author

uncovar/workflow/rules/long_read.smk

Line 85 in e844fc4

rm results/.indicators/replacement_notice.txt

@thomasbtf
Copy link
Contributor

thomasbtf commented Jan 26, 2022 •
edited
Loading

Go catch 👍
This should not have been included.

It does not have to be removed, as the environment should be consitent.

@thomasbtf thomasbtf mentioned this issue Jan 26, 2022
Merged
5 tasks
@thomasbtf
Copy link
Contributor

But just for completeness, what kind of problem did this produce?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thomasbtf thomasbtf

@simakro simakro

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: remove removal of .indicators/replacement_notice.txt for int IKIM-Essen/uncovar
3 participants
@thomasbtf @simakro @alethomas