Skip to content

Suggestions#44

Merged
SPPearce merged 5 commits intodevfrom
suggestions
May 23, 2024
Merged

Suggestions#44
SPPearce merged 5 commits intodevfrom
suggestions

Conversation

@SPPearce
Copy link
Copy Markdown
Contributor

Suggested changes for 1.0.0 release:
Removing the custom process_medium_mem label
Updating bwa/index

@SPPearce SPPearce requested a review from nh13 May 23, 2024 06:07
@github-actions
Copy link
Copy Markdown

github-actions bot commented May 23, 2024

nf-core lint overall result: Passed ✅ ⚠️

Posted for pipeline commit 5865fea

+| ✅ 167 tests passed       |+
#| ❔   8 tests were ignored |#
!| ❗   4 tests had warnings |!
Details

❗ Test warnings:

  • nextflow_config - Config manifest.version should end in dev: 1.0.0
  • readme - README contains the placeholder zenodo.XXXXXXX. This should be replaced with the zenodo doi (after the first release).
  • pipeline_todos - TODO string in README.md: Add citation for pipeline after first release. Uncomment lines below and update Zenodo doi and badge at the top of this file.
  • pipeline_todos - TODO string in output.md: Write this documentation describing your workflow's output

❔ Tests ignored:

✅ Tests passed:

Run details

  • nf-core/tools version 2.14.1
  • Run at 2024-05-23 15:25:31

@SPPearce SPPearce mentioned this pull request May 23, 2024
10 tasks
Copy link
Copy Markdown
Member

@nh13 nh13 left a comment

Choose a reason for hiding this comment

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

Thank-you! LGTM.

@SPPearce SPPearce marked this pull request as ready for review May 23, 2024 12:40
@SPPearce SPPearce merged commit fd404b6 into dev May 23, 2024
@nh13 nh13 deleted the suggestions branch July 17, 2024 10:41
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