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

Error & info messages, code comments language fixes part 1 #3475

Conversation

llewellyn-sl
Copy link
Contributor

No description provided.

@llewellyn-sl llewellyn-sl changed the title Error & info messages, code comments language fixes Error & info messages, code comments language fixes part 1 Dec 6, 2022
Copy link
Member

@abhi18av abhi18av left a comment

Choose a reason for hiding this comment

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

Looking good @llewellyn-sl , are you planning to continue working on this PR w.r.t the entire codebase or this is ready to be merged?

@llewellyn-sl
Copy link
Contributor Author

Looking good @llewellyn-sl , are you planning to continue working on this PR w.r.t the entire codebase or this is ready to be merged?

@abhi18av thanks for reviewing. I'm working my way through the entire codebase, but this is going to be stretched across the next few weeks. I think it would be best to do this in chapters with separate PRs each time.

@pditommaso pditommaso force-pushed the master branch 2 times, most recently from e2b4a93 to f32ea0b Compare December 8, 2022 15:16
Copy link
Member

@pditommaso pditommaso left a comment

Choose a reason for hiding this comment

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

Nice, thanks

@pditommaso pditommaso merged commit 29ae36c into nextflow-io:master Dec 11, 2022
l-modolo pushed a commit to l-modolo/nextflow that referenced this pull request Jan 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants