Skip to content

Conversation

@chasemc
Copy link
Contributor

@chasemc chasemc commented Nov 29, 2023

Replace nf-core code of conduct template with Contributor Covenant template
https://www.contributor-covenant.org/version/2/1/code_of_conduct/code_of_conduct.md

@chasemc chasemc requested a review from jason-c-kwan November 29, 2023 14:01
@chasemc chasemc added the enhancement New feature or request label Nov 29, 2023
@chasemc
Copy link
Contributor Author

chasemc commented Nov 29, 2023

@jason-c-kwan , a contact email needs to go in the spot with [INSERT CONTACT METHOD]. or that part modified

@chasemc
Copy link
Contributor Author

chasemc commented Nov 29, 2023

This is more general and copied from https://www.contributor-covenant.org/

* 🐛 overhaul of GTDB download and setup

moves code to entirely python-based download and setup

* 🐛 overhaul of GTDB download and setup

moves code to entirely python-based download and setup

* 📝 update gtdb download documentation

Removes documentation about autometa-setup-gtdb, which can be revisited later.

That pathway makes it hard to ensure reproducibility since the files, as downloaded, don't have version info. For now, for the sake of ensuring reproducibility, the only thing accepted are Autometa-downloaded files.

* 🐛 make compatible with pre-Python version 3.9

* 🐛 fix gtdb download tests

* add back shebang

* ensure gtdb directories exist

* change default GTDB host url

* fix GTDB base url

* fix url

* prevent GTDB versions <220

* update documentation to reflect version 220 or greater

* bump action versions
Replace nf-core code of conduct template with Contributor Covenant template
https://www.contributor-covenant.org/version/2/1/code_of_conduct/code_of_conduct.md
@chasemc
Copy link
Contributor Author

chasemc commented Dec 10, 2024

moved to #368

@chasemc chasemc closed this Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants