8000 ci: Move tmpdir to ~ by edmundmiller · Pull Request #791 · nf-core/sarek · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

ci: Move tmpdir to ~ #791

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 8000

Jump to bottom
Merged
merged 1 commit into from
Oct 11, 2022
Merged

ci: Move tmpdir to ~ #791

merged 1 commit into from
Oct 11, 2022

Conversation

edmundmiller
Copy link
Contributor

Changes the pytest workflow tmpdir to ~ (which is already done in the test_all)

@edmundmiller @awgymer
ci: Move tmpdir to ~
50b8f7b 
Co-authored-by: awgymer <awgymer@users.noreply.github.com>
@edmundmiller edmundmiller self-assigned this Oct 11, 2022
@edmundmiller edmundmiller changed the base branch from master to dev October 11, 2022 14:06
@nf-core nf-core deleted a comment from github-actions bot Oct 11, 2022
@nf-core nf-core deleted a comment from github-actions bot Oct 11, 2022
@FriederikeHanssen
Copy link
Contributor

why is this better?

@FriederikeHanssen FriederikeHanssen merged commit ef79ba0 into nf-core:dev Oct 11, 2022
@edmundmiller edmundmiller deleted the tmpdir branch October 11, 2022 14:14
@edmundmiller
Copy link
Contributor Author

Idk it's been that way in modules for a while 🙃

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

@maxulysse maxulysse maxulysse approved these changes

@FriederikeHanssen FriederikeHanssen FriederikeHanssen approved these changes

Assignees

@edmundmiller edmundmiller

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@edmundmiller @FriederikeHanssen @maxulysse
0