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

Add bug fixes from master to v1.2.0 #52

Merged
merged 19 commits into from
Apr 11, 2024
Merged

Add bug fixes from master to v1.2.0 #52

merged 19 commits into from
Apr 11, 2024

Conversation

dthoward96
Copy link
Collaborator

No description provided.

rchau88 and others added 19 commits February 9, 2024 17:35
* Delete .github/workflows/codeql-analysis.yml

* Delete .github/workflows/python-package-mamba.yml
Missing a return for config file function and correcting a error message to print the correct df
* Create GHCR_docker.yml

* Update GHCR_docker.yml

Correct changes for master branch
Some FTP accounts have the folder structure /submit/Production/ instead of /Production/. This fix automatically corrects for this difference in folder structure.
update template metadata with required fields for flu
removed the gs-sequence_name specified for flu
automatic test builds dockerfile on pull request. This will prevent merging to master if Dockerfile fails to build correctly.
* Update cov_config.yaml

remove org_id from examples

* Update flu_config.yaml

remove org_id from examples

* Update create.py

remove org_id from xml creation
* Update report.py

Bug fix to allow for other submit folders in FTP of ncbi

* Update process.py

bug fix for capitalization of folder name
Changes requirement from only isolate to require either strain or isolate for BioSample and GenBank
Fixes issue that creates duplicate strain columns when using src-strain
Bug fix for upload log. When only one database was submitted it, it would convert the database name into a list and error out.
@dthoward96 dthoward96 merged commit ea50744 into v1.2.0 Apr 11, 2024
4 checks passed
@dthoward96
Copy link
Collaborator Author

had to rebuild cache after branch naming issue

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