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

feat(ena-submission): Add a dry run option for faster debugging/ submission review/ updates #2931

Merged
merged 5 commits into from
Oct 4, 2024

Conversation

anna-parker
Copy link
Contributor

@anna-parker anna-parker commented Oct 2, 2024

resolves #

preview URL: https://ena-deposition-add-dry-ru.loculus.org/

Summary

Screenshot

PR Checklist

  • All necessary documentation has been adapted.
  • The implemented feature is covered by an appropriate test.

@anna-parker anna-parker changed the title Add a dry run option for faster debugging/ submission review/ updates feat(ena-submission): Add a dry run option for faster debugging/ submission review/ updates Oct 2, 2024
Copy link
Contributor

@corneliusroemer corneliusroemer left a comment

Choose a reason for hiding this comment

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

nice

@anna-parker anna-parker added the preview Triggers a deployment to argocd label Oct 4, 2024
@anna-parker anna-parker marked this pull request as ready for review October 4, 2024 07:47
@anna-parker
Copy link
Contributor Author

Tested locally - this could still use some refactoring but will merge to make future debugging easier.

ena-submission/Snakefile Outdated Show resolved Hide resolved
@anna-parker anna-parker removed the preview Triggers a deployment to argocd label Oct 4, 2024
@anna-parker anna-parker merged commit 91a5115 into main Oct 4, 2024
15 checks passed
@anna-parker anna-parker deleted the ena_deposition_add_dry_run_option branch October 4, 2024 08: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.

2 participants