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

Reconsider process resource declarations in config files #66

Closed
boulund opened this issue Mar 19, 2018 · 2 comments
Closed

Reconsider process resource declarations in config files #66

boulund opened this issue Mar 19, 2018 · 2 comments
Labels
enhancement Low priority
Milestone
BACTpipe 3.0

Comments

@boulund
Copy link
Member

boulund commented Mar 19, 2018 •
edited

I suggest this change due to a recent commit to Nextflow: nextflow-io/nextflow#623

This will make the config files a bit cleaner overall. As the old configuration notation (e.g. process.$processName.cpu) is deprecated, we probably should change over eventually.
@boulund boulund modified the milestones: Bactpipe 2.0, Bactpipe 3.0 Mar 29, 2018
@boulund boulund removed the quick fix label Mar 29, 2018
@boulund
Copy link
Member Author

boulund commented Mar 29, 2018

Changed this to a 3.0 milestone and removed the quick fix label. This is not important enough to warrant work in the 2.0 milestone.

@boulund
Copy link
Member Author

boulund commented Jan 16, 2019

I got annoyed with the warnings about using deprecated syntax so I dealt with this now. Will merge to develop in a few minutes

@boulund boulund closed this as completed Jan 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement Low priority
Projects
None yet
Milestone
BACTpipe 3.0
Development

No branches or pull requests
1 participant
@boulund