Skip to content
This repository was archived by the owner on Aug 20, 2024. It is now read-only.

Conversation

@mirpedrol
Copy link
Collaborator

No description provided.

@mirpedrol mirpedrol requested review from ewels and nvnieuwk October 12, 2023 06:55
Copy link
Member

@ewels ewels left a comment

Choose a reason for hiding this comment

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

Some minor suggested additions to the changelog, otherwise LGTM 👍🏻

Co-authored-by: Phil Ewels <phil.ewels@seqera.io>
Copy link
Collaborator

@nvnieuwk nvnieuwk left a comment

Choose a reason for hiding this comment

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

Can you also bump the manifest here?

@mirpedrol
Copy link
Collaborator Author

Can you also bump the manifest here?

Done here :) or do you mean something else?

@nvnieuwk
Copy link
Collaborator

Oh yup my bad 😁

Copy link
Collaborator

@nvnieuwk nvnieuwk 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 merged commit ffa4134 into nextflow-io:master Oct 12, 2023
@mirpedrol mirpedrol deleted the bump-1.0.0 branch October 12, 2023 08:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants