Skip to content

Conversation

@Bisaloo
Copy link
Contributor

@Bisaloo Bisaloo commented Aug 25, 2025

To make it easier to find a specific option

If you are submitting a pull request to BiocCheck please follow the
instructions.
The presentation includes steps for forking, creating a working branch, and
useful related information.

For a successful merge, the following steps are required:

  • Update the NEWS file
  • Update the vignette file
  • Add unit tests (optional but highly recommended)
  • Passing R CMD build & R CMD check on Bioconductor devel

List a reviewer in the Pull Request (either @LiNk-NY, @lshep, @mtmorgan).
Reviewers will make sure to Comment, Approve or Request changes.

We highly recommend the use of the Bioconductor devel docker image described
on the Bioconductor website.

If you have any questions, please get in touch with the Bioconductor core team
on the Bioconductor Community Slack.

Copy link
Contributor

@LiNk-NY LiNk-NY 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 for the PR!
There are a couple of minor changes that would get this through.

@codecov
Copy link

codecov bot commented Aug 25, 2025

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@LiNk-NY LiNk-NY merged commit dc64071 into Bioconductor:devel Aug 25, 2025
3 of 4 checks passed
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