Change "insights" to "quarkus-insights" and remove tag of "quarkus" - #2832
Change "insights" to "quarkus-insights" and remove tag of "quarkus"#2832insectengine wants to merge 17 commits into
Conversation
…rkus" For discoverability and clarity, I'm changing the "insights" tag to "quarkus-insights" on 17 posts. I'm also removing an orphan "quarkus" tag that doesn't need to exist.
This comment has been minimized.
This comment has been minimized.
|
I was briefly puzzled by the failures in this PR, then I worked out what was happening. It looks like the checks of #2713, but #2713 isn't merged. And I didn't know why #2713 was finding non-fragment failures. I assumed it had accidentally fixed a bug in the crawler. But no. It's that we happen to have blogs on the front page now (ie recent blogs) which expose the pre-existing tag problem fixed by #2831. This PR causes the link checker to run on the front page, which it wouldn't always, so it is failing with the issue. |
holly-cummins
left a comment
holly-cummins
left a comment
There was a problem hiding this comment.
If we wanted to be really fancy we'd add a redirect from /tags/insights to /tags/quarkus-insights, but I don't know if anything would ever have linked to the old url.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Runs Lighthouse against the test server after smoke tests, scoring performance, accessibility, best practices, and SEO for key pages. Results appear as a color-coded table in the GitHub step summary and are uploaded as build artifacts. Uses npm ci with a lockfile to pin @lhci/cli dependencies. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> # Conflicts: # .github/workflows/build.yml
Keeps @lhci/cli and any future npm dependencies up to date. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Reuses the link_scope step outputs: skips when no HTML changed, tests only changed pages (capped at 5) for partial PRs, and falls back to key pages (/, /guides/, /blog/) for full checks. URLs are now passed via CLI args instead of hardcoded in lighthouserc.js. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Reads report URLs from .lighthouseci/links.json (written by the temporary-public-storage upload) and adds them to the step summary table. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This comment has been minimized.
This comment has been minimized.
|
🙈 The PR is closed and the preview is expired. |
Status for workflow
|
|
Was easier to just close this PR and redo the work and resubmit to avoid the unnecessary commits. |
3 participants
For discoverability and clarity, I'm changing the "insights" tag to "quarkus-insights" on 17 posts. I'm also removing an orphan "quarkus" tag that doesn't need to exist.
** If you are updating a guide, please submit your pull request to the main repository: https://github.com/quarkusio/quarkus/tree/main/docs/src/main/asciidoc **