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.

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

Make synonym outputs explicit inputs to other rules in Snakemake #124

Merged
merged 3 commits into from
May 11, 2023

Conversation

gaurav
Copy link
Collaborator

@gaurav gaurav commented Apr 18, 2023

This PR adds the synonym files as inputs to the final completion task. This allows them to be regenerated simply by deleting them and re-running snakemake.

Should be merged after PR #130.

@gaurav gaurav force-pushed the make-synonym-outputs-explicit branch from 32372a1 to c6884c7 Compare April 19, 2023 01:50
@gaurav gaurav changed the base branch from master to synonym_prototype May 5, 2023 22:13
@gaurav gaurav marked this pull request as ready for review May 6, 2023 20:07
@gaurav gaurav marked this pull request as draft May 6, 2023 20:07
@gaurav gaurav mentioned this pull request May 8, 2023
Base automatically changed from synonym_prototype to master May 8, 2023 01:14
@gaurav gaurav force-pushed the make-synonym-outputs-explicit branch from a353a10 to a0b92f7 Compare May 8, 2023 01:20
@gaurav gaurav changed the base branch from master to fix-ncbigene-synonym-problem May 8, 2023 01:36
@gaurav gaurav force-pushed the make-synonym-outputs-explicit branch 2 times, most recently from 5abfb7d to aa96528 Compare May 8, 2023 01:39
@gaurav gaurav requested a review from cbizon May 8, 2023 01:40
@gaurav gaurav marked this pull request as ready for review May 8, 2023 01:40
Base automatically changed from fix-ncbigene-synonym-problem to master May 11, 2023 15:16
@gaurav gaurav merged commit 031f64d into master May 11, 2023
@gaurav gaurav deleted the make-synonym-outputs-explicit branch May 11, 2023 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants