Skip to content

Conversation

@marc-jones
Copy link
Contributor

luslab-nf-modules

Thank you for contributing and submitting a pull request!

Please check you have done the following:

Uniform coding style

  • Used snake_case rather than camelCase
  • Followed the output channel naming convention (e.g. bam for BAM files, bed for BED files)
  • Set the resource label for a new process

Continuous integration

  • Added any new modules to .github/workflows/modules-testing.yml
  • Added any new containers to .github/workflows/docker-build-push.yml and .github/workflows/docker-linting.yml
  • Added any new workflows to .github/workflows/workflow-testing.yml
  • Added any new test data to test_data
  • Bumped the version when changing an existing module
    • Major.Minor.Patch
      • Major for breaking changes
      • Minor for feature updates and most things
      • Patch for bug fixes and small things like comment and variable name updates

Review

  • Selected some reviewers
  • Assigned yourself to the PR
  • Tagged the PR appropriately

@marc-jones marc-jones added the enhancement New feature or request label Apr 25, 2022
@marc-jones marc-jones self-assigned this Apr 25, 2022
@marc-jones marc-jones marked this pull request as draft April 25, 2022 12:25
@marc-jones marc-jones changed the title Update syntax for all modules Update syntax for most modules Apr 25, 2022
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.

3 participants