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

Re-organize configuration files to match pathogen repo guide [#27] #28

Merged
merged 2 commits into from
Jul 11, 2024

Conversation

genehack
Copy link
Contributor

@genehack genehack commented Jul 11, 2024

Description of proposed changes

Rename configuration files and adjust paths within other files to match.[#27]

  • mv ingest/config -> ingest/defaults
  • mv phylogenetic/config -> phylogenetic/defaults
  • mv ingest/config/defaults.yaml -> ingest/defaults/config.yaml
  • mv phylogenetic/config/defaults.yaml -> phylogenetic/defaults/config.yaml

Add convenience clean rule to both Snakefiles [#27]

Related issue(s)

#27

Checklist

  • Checks pass

* mv ingest/config -> ingest/defaults
* mv phylogenetic/config -> phylogenetic/defaults
* mv ingest/config/defaults.yaml -> ingest/defaults/config.yaml
* mv phylogenetic/config/defaults.yaml -> phylogenetic/defaults/config.yaml

Adjust paths within other files to match.
@genehack genehack requested a review from a team July 11, 2024 00:23
Copy link

@j23414 j23414 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@genehack genehack merged commit b412fcd into main Jul 11, 2024
6 checks passed
@genehack genehack deleted the re-org-config-27 branch July 11, 2024 17:15
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.

3 participants