Skip to content

Link the GHA optimization page from the Optimize PR workflows tip - #321

Merged
cottage14 merged 2 commits into
apache:masterfrom
potiuk:link-gha-optimization-page
Oct 24, 2026
Merged

cottage14 merged 2 commits into
apache:masterfrom
potiuk:link-gha-optimization-page

Conversation

@potiuk

@potiuk potiuk commented Oct 23, 2026

Copy link
Copy Markdown
Member

Adds a link to the new Optimizing GitHub Actions workflows page in the Optimize PR workflows tip on the Services and Tools page, pointing at the selective-tests and canary-build sections that match what the tip describes.

🤖 Generated with Claude Code

@potiuk

potiuk commented Oct 23, 2026

Copy link
Copy Markdown
Member Author

:) :)

@potiuk

potiuk commented Oct 24, 2026

Copy link
Copy Markdown
Member Author

Green now. Ready to merge :)

@potiuk

potiuk commented Oct 24, 2026

Copy link
Copy Markdown
Member Author

Failed linting due to markdownlint errors - mostly spaces at the end of line - in the last newsletter - I fixed them all - I can also separate those fixes out if needs be.

@cottage14

Copy link
Copy Markdown
Contributor

Failed linting due to markdownlint errors - mostly spaces at the end of line - in the last newsletter - I fixed them all - I can also separate those fixes out if needs be.

I need to reject the corrections that change italics to boldface. I have asked Justin where the settings are to turn off that option,

@cottage14 cottage14 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A bit of a shakedown cruise for linting. It requires changes that are not necessary, and others that make it harder for a human to review and edit the document. I have asked Justin where the controls are for the linter so I can turn off the options that do not add value

@potiuk

potiuk commented Oct 24, 2026

Copy link
Copy Markdown
Member Author

I can implement those without asking :)

@cottage14 cottage14 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Although I find some of the forced changes to satisfy linting annoying, they do not actually break the page display. I will approve this, and in the coming week see what I can do about the linter settings.

@cottage14
cottage14 merged commit da8c0a8 into apache:master Oct 24, 2026
1 check passed
@potiuk

potiuk commented Oct 24, 2026

Copy link
Copy Markdown
Member Author

Here is PR that fixes it according to your liking @cottage14 #322

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