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

Bump jekyll-generator-single-source #1623

Merged
merged 29 commits into from
Nov 7, 2024

Conversation

fabianrbz
Copy link
Contributor

Update the site to use the new version of jekyll-generator-single-source.

Most notable changes:

  • we added support for labeling releases
  • URLs now use /{{ page.release }}/ instead of /{{ page.version }} which is more accurate
  • the way to handle dev release changes slightly, instead of having a app/_data/docs_nav_dev.yml it needs to be named after the next release, e.g. app/_data/docs_nav_2.6.x.yml. The dev bit is handled via the app/_data/versions.yml file, by adding label: dev to the corresponding version.

This also updates:

  • ruby + dependencies
  • simplifies the way to run the app locally make clean && make run builds and runs the site, including redirects

Did you sign your commit? Instructions

Have you read Contributing guidelines?

Copy link

netlify bot commented Jan 29, 2024

Deploy Preview for kuma ready!

Name Link
🔨 Latest commit 6f2fbcc
🔍 Latest deploy log https://app.netlify.com/sites/kuma/deploys/672cb509ad3d910008fee55d
😎 Deploy Preview https://deploy-preview-1623--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: 85 (🟢 up 2 from production)
Accessibility: 90 (no change from production)
Best Practices: 100 (no change from production)
SEO: 92 (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.

@lahabana
Copy link
Contributor

@fabianrbz can you rebase seems like there are some conflicts

@fabianrbz fabianrbz force-pushed the refactor-version-is branch 3 times, most recently from 3f62a48 to 6215139 Compare February 1, 2024 12:13
@fabianrbz
Copy link
Contributor Author

@lahabana test is failing due to a caching issue, can you clear the cache for that action?

@fabianrbz fabianrbz force-pushed the refactor-version-is branch 2 times, most recently from a670374 to df90f88 Compare February 23, 2024 15:33
@fabianrbz fabianrbz marked this pull request as ready for review February 26, 2024 08:11
@fabianrbz fabianrbz requested a review from a team as a code owner February 26, 2024 08:11
@fabianrbz fabianrbz requested review from Automaat and bartsmykla and removed request for a team February 26, 2024 08:11
@lahabana
Copy link
Contributor

lahabana commented Mar 7, 2024

@fabianrbz is this ready to review?

@fabianrbz
Copy link
Contributor Author

@lahabana yes

@slonka
Copy link
Contributor

slonka commented Mar 28, 2024

@fabianrbz - tests are failing, can you look into them?

@fabianrbz fabianrbz changed the title Refactor version is Bump jekyll-generator-single-source Mar 28, 2024
@lahabana
Copy link
Contributor

@fabianrbz are you still looking into this?

1 similar comment
@slonka
Copy link
Contributor

slonka commented Aug 14, 2024

@fabianrbz are you still looking into this?

@fabianrbz
Copy link
Contributor Author

@lahabana @slonka sorry, I'm not working on this atm, I might be able to find some time after summit though

Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Remove jekyll-plugins section, list them in the jekyll config file

Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Skip versions that are labelled with `dev`

Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
The generators defined here should have a lower priority than the one
defined in jekyll-generator-single-source. Also, medium isn't a valid
priority.

Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Signed-off-by: Fabian Rodriguez <fabian.rodriguez@konghq.com>
Signed-off-by: Charly Molter <charly.molter@konghq.com>
Signed-off-by: Charly Molter <charly.molter@konghq.com>
Signed-off-by: Charly Molter <charly.molter@konghq.com>
app/_includes/version_alert.html Outdated Show resolved Hide resolved
app/_layouts/default.html Show resolved Hide resolved
Signed-off-by: Charly Molter <charly.molter@konghq.com>
@lahabana lahabana merged commit f9202ba into kumahq:master Nov 7, 2024
15 of 16 checks passed
lahabana added a commit to kumahq/kuma that referenced this pull request Nov 8, 2024
This fixes: kumahq/kuma-website#2073 which is needed as we moved to the latest site-generator with: kumahq/kuma-website#1623

Signed-off-by: Charly Molter <charly.molter@konghq.com>
lahabana added a commit to kumahq/ci-tools that referenced this pull request Nov 8, 2024
With kumahq/kuma-website#1623 all sites
use `label:dev` so it's no longer needed to have a flag for this

Signed-off-by: Charly Molter <charly.molter@konghq.com>
lahabana added a commit to kumahq/ci-tools that referenced this pull request Nov 8, 2024
With kumahq/kuma-website#1623 all sites
use `label:dev` so it's no longer needed to have a flag for this

Signed-off-by: Charly Molter <charly.molter@konghq.com>
lahabana added a commit to kumahq/kuma that referenced this pull request Nov 8, 2024
## Motivation

This fixes: kumahq/kuma-website#2073 which is
needed as we moved to the latest site-generator with:
kumahq/kuma-website#1623.

---------

Signed-off-by: Charly Molter <charly.molter@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.

3 participants