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

feat(app/nav): add 2.9.x #1972

Merged
merged 2 commits into from
Oct 15, 2024

Conversation

schogges
Copy link
Contributor

@schogges schogges commented Oct 15, 2024

Adds navigation for release-2.9.x and fixes 404 on /docs/2.9.x.
Also fixes ci error in Docs lint due to error: externally-managed-environment on ubuntu ubuntu-24.04 when installing docutils, by pin ubuntu to 22.04.

Signed-off-by: schogges <moritz.fleck@konghq.com>
Copy link

netlify bot commented Oct 15, 2024

Deploy Preview for kuma ready!

Name Link
🔨 Latest commit 31dbd03
🔍 Latest deploy log https://app.netlify.com/sites/kuma/deploys/670e5743c4a8560008721fa4
😎 Deploy Preview https://deploy-preview-1972--kuma.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 65 (🔴 down 3 from production)
Accessibility: 90 (no change from production)
Best Practices: 100 (no change from production)
SEO: 85 (no change from production)
PWA: 80 (no change from production)
View the detailed breakdown and full score reports

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

@schogges schogges marked this pull request as ready for review October 15, 2024 11:36
@schogges schogges requested a review from a team as a code owner October 15, 2024 11:36
@schogges schogges requested review from slonka and lukidzi and removed request for a team October 15, 2024 11:36
@schogges schogges force-pushed the feat/app/_data/docs_nav_kuma_2-9-x branch from ffd7a06 to 9b53270 Compare October 15, 2024 11:48
Signed-off-by: schogges <moritz.fleck@konghq.com>
@schogges schogges force-pushed the feat/app/_data/docs_nav_kuma_2-9-x branch from 9b53270 to 31dbd03 Compare October 15, 2024 11:51
Copy link
Contributor

@johncowen johncowen left a comment

Choose a reason for hiding this comment

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

Thanks!

@johncowen johncowen merged commit f1cf7d7 into kumahq:master Oct 15, 2024
16 checks passed
@schogges schogges deleted the feat/app/_data/docs_nav_kuma_2-9-x branch October 15, 2024 12:03
johncowen added a commit that referenced this pull request Oct 17, 2024
…1975)

Back in #1972 following a suggested fix for a python error in the  vale GH action, we pinned ubuntu to what seemed like a working version. We've since seen that whilst it looked to be working, there were still errors deeper in the logs that somehow weren't caught by CI.

This PR:

1. Bumps the version of the vale GH action (I'm hoping/assuming this is why these errors weren't caught in CI?)
2. Changes the location of the Base vocabularies as a consequence of the GH action using `latest` for the version of vale to use i.e. vale version 3.something. The previous location was only supported in vale 2.something. This change fixed an error that has been appearing in CI logs over several PRs.
3. Puts things back to use ubuntu latest


Closes: #1973

---------

Signed-off-by: John Cowen <john.cowen@konghq.com>
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