Skip to content

Blindly auto-fix all headings #617

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

Merged
merged 5 commits into from
Jun 4, 2025
Merged

Conversation

ewels
Copy link
Member

@ewels ewels commented Jun 4, 2025

There seem to be some dodgy headings that are now causing CI to fail after #596 is merged, if those files are touched.

Ran the auto-fix script to clean them up here, but might need some oversight - especially if some of these are affected by drift between English / translated versions.

Copy link

netlify bot commented Jun 4, 2025

Deploy Preview for nextflow-training ready!

Name Link
🔨 Latest commit b65f14d
🔍 Latest deploy log https://app.netlify.com/projects/nextflow-training/deploys/68404cf2e905930008dc41b6
😎 Deploy Preview https://deploy-preview-617--nextflow-training.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@ewels ewels mentioned this pull request Jun 4, 2025
@vdauwera vdauwera self-requested a review June 4, 2025 13:25
vdauwera
vdauwera previously approved these changes Jun 4, 2025
Copy link
Collaborator

@vdauwera vdauwera left a comment

Choose a reason for hiding this comment

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

There were a few wrong fixes where the right thing to do was ambiguous. Fixed them all as far as I can tell, should be good to go.

@vdauwera
Copy link
Collaborator

vdauwera commented Jun 4, 2025

Actually I caught a few more, nobody merge until i finish fixing this

@vdauwera vdauwera self-requested a review June 4, 2025 13:48
Copy link
Collaborator

@vdauwera vdauwera left a comment

Choose a reason for hiding this comment

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

I think I got them all this time.

@vdauwera vdauwera merged commit 7a111d4 into nextflow-io:master Jun 4, 2025
6 checks passed
@vdauwera vdauwera added bug Something is wrong or not working documentation Improvements or additions to documentation Translations Support for non-English languages hello series labels Jun 4, 2025
@ewels ewels deleted the fix-all-headings branch June 4, 2025 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is wrong or not working documentation Improvements or additions to documentation hello series Translations Support for non-English languages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants