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.

Sign up for GitHub

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

Jump to bottom

Release 1.0.0 #51

Merged
merged 341 commits into from
Jun 11, 2024
Merged

Release 1.0.0 #51

merged 341 commits into from
Jun 11, 2024

Conversation

itrujnara
Copy link
Collaborator

This is the first release of nf-core/reportho.

PR checklist

  • This comment contains a description of changes (with reason).
  • If you've fixed a bug or added code that should be tested, add tests!
  • If you've added a new tool - have you followed the pipeline conventions in the contribution docs
  • If necessary, also make a PR on the nf-core/reportho branch on the nf-core/test-datasets repository.
  • Make sure your code lints (nf-core lint).
  • Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
  • Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
  • Usage Documentation in docs/usage.md is updated.
  • Output Documentation in docs/output.md is updated.
  • CHANGELOG.md is updated.
  • README.md is updated (including new tool citations and authors/contributors).

itrujnara and others added 30 commits May 2, 2024 11:09
@itrujnara
Switched parameters to be false by default
2bac397
@itrujnara
Added copyright info to scripts
a70c77b
@itrujnara
Missing copyright note
1abec30
@itrujnara
Removed a fulfilled TODO
f960b85
@JoseEspinosa
Get rid of directory probably added by mistake
a1e8c38
@itrujnara @JoseEspinosa
Update bin/yml2csv.py
fbfbde6 
Co-authored-by: Jose Espinosa-Carrasco <[email protected]>
@itrujnara
Added FASTA example to the usage document
cf3d0dd
@itrujnara
Merge branch 'dev' of https://github.com/itrujnara/reportho into dev
f848a1f
@itrujnara
Refactored Orthoinspector parameter names
8fc2254
@JoseEspinosa
Merge pull request #13 from itrujnara/dev
61581ea 
Multiple tweaks
@JoseEspinosa
Add missing containers to local modules
2e2145e
@JoseEspinosa
Get rid of params inside module
c85b7f4
@JoseEspinosa
Add stubs to local modules
e1f6d07
@itrujnara
Linting fix - subworkflow versions
f59ba77
@itrujnara
Version bump to 1.0.0
45e46c2
@itrujnara
Removed dev note from readme
235a872
@itrujnara
Bug fix - React version reporting
44c8348
@itrujnara
Updated changelog
8d0dc11
@itrujnara
Merge branch 'dev' of https://github.com/itrujnara/reportho into dev
0bb2373
@JoseEspinosa
Fix typos
eb7fe03
@JoseEspinosa
Only run when local FETCH_EGGNOG_GROUP_LOCAL
baff9b9
@JoseEspinosa
Add stub
266ff2d
@JoseEspinosa
Update test.config
8a6b709
@JoseEspinosa
Merge remote-tracking branch 'upstream/dev' into updates
e6dbdbc
@JoseEspinosa
Make lint happy
f7d9e68
@JoseEspinosa
Fix parameters for REPORT
dc14ae9
@JoseEspinosa
Fix nf-core lint
f5d3637
@JoseEspinosa
Add stub
4cb7aaa
@JoseEspinosa
Fix test
9aae4de
@JoseEspinosa
Force update of utils_nfcore_pipeline to make nf-core lint happy
7a3d3ac
ramprasadn
ramprasadn approved these changes Jun 7, 2024
View reviewed changes
Copy link

@ramprasadn ramprasadn left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

mirpedrol
mirpedrol approved these changes Jun 7, 2024
View reviewed changes
Copy link
Member

@mirpedrol mirpedrol left a comment

Choose a reason for hiding this comment

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

LGTM!

@itrujnara itrujnara merged commit dd05afe into master Jun 11, 2024
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ramprasadn ramprasadn ramprasadn approved these changes

@mirpedrol mirpedrol mirpedrol approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@itrujnara @mirpedrol @ramprasadn @JoseEspinosa @nf-core-bot @luisas