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.

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

Release v2.1.0 #26

Merged
merged 100 commits into from
Apr 2, 2024
Merged

Release v2.1.0 #26

merged 100 commits into from
Apr 2, 2024

Conversation

melferink
Copy link
Member

@melferink melferink commented Mar 27, 2024

Copy link
Collaborator

@ellendejong ellendejong left a comment

Choose a reason for hiding this comment

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

Most commits have been approved previously.
Except 91258ad, c9a7fdb, dd4391d: OK.

However, not all files are conform flake8 which is required.

ClarityEpp/SampleUDFDx.nf Show resolved Hide resolved
GenderCheck/calculate_gender.py Show resolved Hide resolved
GenderCheck/test_calculate_gender.py Outdated Show resolved Hide resolved
MosaicHunter/1.0.0/Dockerfile Outdated Show resolved Hide resolved
@melferink melferink requested a review from rernst April 2, 2024 08:40
Copy link
Member

@rernst rernst left a comment

Choose a reason for hiding this comment

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

Comments by @ellendejong are taken care of.

@rernst rernst dismissed ellendejong’s stale review April 2, 2024 08:44

Comments by @ellendejong are taken care of.

@melferink melferink merged commit 5ae3239 into main Apr 2, 2024
@BasMonkey BasMonkey deleted the release/v2.1.0 branch April 15, 2024 07:48
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.

4 participants