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

package: upgrade eleventy #790

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

package: upgrade eleventy #790

wants to merge 1 commit into from

Conversation

nizamial09
Copy link
Member

Following the eleventy upgrade helper to take care of the upgrade: https://www.11ty.dev/docs/plugins/upgrade-help/

Upgrade to eleventy@1

Generated the package-lock using node 18 and didn't see any major issue
there.

Only notice the upgrade helper pointed out which is valid for us is the
transition from slug to slugify. But they disregarded this as an
optional because slug will continue to work for universal filter
https://www.11ty.dev/docs/filters/slugify/#changing-slug-to-slugify

@11ty/eleventy-upgrade-help] NOTICE `slug` to `slugify` filter mismatches found. No action is strictly required here, but if this site has been published publicly make sure that you continue to use the `slug` filter for these templates or your public facing URLs will change/break!

'anitab.org'
Expected values to be strictly equal:
+ actual - expected

+ 'anitab.org'
- 'anitab-org'
         ^

'wcag2.1'
Expected values to be strictly equal:
+ actual - expected

+ 'wcag2.1'
- 'wcag2-1'
        ^

"Fira Barcelona Gran Via, Av. Joan Carles I, 64, 08908 L'Hospitalet de Llobregat, Barcelona, Spain-Barcelona, Spain"
Expected values to be strictly equal:
+ actual - expected

+ "fira-barcelona-gran-via-av.-joan-carles-i-64-08908-l'hospitalet-de-llobregat-barcelona-spain-barcelona-spain"
- 'fira-barcelona-gran-via-av-joan-carles-i-64-08908-l-hospitalet-de-llobregat-barcelona-spain-barcelona-spain'

'undefined-https://meet.jit.si/ceph-user-dev-monthly'
Expected values to be strictly equal:
+ actual - expected

+ 'undefined-https:meet.jit.siceph-user-dev-monthly'
- 'undefined-https-meet-jit-si-ceph-user-dev-monthly'
                  ^

'null-CERN | Geneva, Switzerland'
Expected values to be strictly equal:
+ actual - expected

+ 'null-cern-or-geneva-switzerland'
- 'null-cern-geneva-switzerland'
             ^

Upgrade to eleventy@2

Major change:
The default dev server is now Eleventy Dev Server https://www.11ty.dev/docs/dev-server/.
But we still can use the BrowserSync as they stated in the doc
https://www.11ty.dev/docs/dev-server/#swap-back-to-browsersync

@nizamial09 nizamial09 force-pushed the update-eleventy branch 2 times, most recently from 1d69d6a to 89750cd Compare February 3, 2025 09:36
Generated the package-lock using node 18 and didn't see any major issue
there.

Following the eleventy upgrade helper to take care of the upgrade: https://www.11ty.dev/docs/plugins/upgrade-help/

Only notice the upgrade helper pointed out which is valid for us is the
transition from `slug` to `slugify`. But they disregarded this as an
optional because `slug` will continue to work for universal filter
https://www.11ty.dev/docs/filters/slugify/#changing-slug-to-slugify

```
@11ty/eleventy-upgrade-help] NOTICE `slug` to `slugify` filter mismatches found. No action is strictly required here, but if this site has been published publicly make sure that you continue to use the `slug` filter for these templates or your public facing URLs will change/break!

'anitab.org'
Expected values to be strictly equal:
+ actual - expected

+ 'anitab.org'
- 'anitab-org'
         ^

'wcag2.1'
Expected values to be strictly equal:
+ actual - expected

+ 'wcag2.1'
- 'wcag2-1'
        ^

"Fira Barcelona Gran Via, Av. Joan Carles I, 64, 08908 L'Hospitalet de Llobregat, Barcelona, Spain-Barcelona, Spain"
Expected values to be strictly equal:
+ actual - expected

+ "fira-barcelona-gran-via-av.-joan-carles-i-64-08908-l'hospitalet-de-llobregat-barcelona-spain-barcelona-spain"
- 'fira-barcelona-gran-via-av-joan-carles-i-64-08908-l-hospitalet-de-llobregat-barcelona-spain-barcelona-spain'

'undefined-https://meet.jit.si/ceph-user-dev-monthly'
Expected values to be strictly equal:
+ actual - expected

+ 'undefined-https:meet.jit.siceph-user-dev-monthly'
- 'undefined-https-meet-jit-si-ceph-user-dev-monthly'
                  ^

'null-CERN | Geneva, Switzerland'
Expected values to be strictly equal:
+ actual - expected

+ 'null-cern-or-geneva-switzerland'
- 'null-cern-geneva-switzerland'
             ^
```
Signed-off-by: Nizamudeen A <[email protected]>
@nizamial09 nizamial09 marked this pull request as ready for review February 3, 2025 10:23
@nizamial09
Copy link
Member Author

I generated the package files using node 23. I also updated the jenkins job (temporarily) to run with an updated node.js version. The latest I could find on debian is 20.x. I am updating the build script officially here: ceph/ceph-build#2320

https://jenkins.ceph.com/job/ceph-website-prs/1118/consoleFull

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.

1 participant