Skip to content

Conversation

@DerekNonGeneric
Copy link
Member

Fixes: #1462

This is the whole work order of migrating Jekyll to Eleventy.

@DerekNonGeneric DerekNonGeneric added 📦 Type: Dependencies PRs that modify packages req'd by product in production 🈺 Type: Meta PRs or issues regarding the general management of the project 🟨 Code: JS/TS PRs or issues regarding JavaScript/TypeScript code 🏗️ Category: Build Issues and PRs related to the build directory 📦 Type: Developer Dependencies PRs that modify packages only req'd for local development and testing 🗞️ Category: Blog undefined 🏗️ Category: Infrastructure/Tooling/Builds/CI undefined ✨ Type: Enhancement undefined ⚙️ Category: Engineering Problems/updates involving the functionality of the product labels Oct 27, 2024
@netlify
Copy link

netlify bot commented Oct 27, 2024

Deploy Preview for gh-pages-openinf ready!

Name Link
🔨 Latest commit e508152
🔍 Latest deploy log https://app.netlify.com/sites/gh-pages-openinf/deploys/671de913920bba0008b71d9f
😎 Deploy Preview https://deploy-preview-1476--gh-pages-openinf.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@socket-security
Copy link

socket-security bot commented Oct 27, 2024

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@11ty/eleventy@3.0.0 environment Transitive: eval, filesystem, network, shell, unsafe +161 18.1 MB zachleat

🚮 Removed packages: npm/strip-comments@2.0.1, npm/stylelint-config-recess-order@5.1.1, npm/stylelint-config-standard-scss@13.1.0, npm/stylelint@16.10.0, npm/tsx@4.19.1, npm/typescript@5.6.3

View full report↗︎

DerekNonGeneric and others added 2 commits October 26, 2024 22:36
Signed-off-by: Derek Lewis <DerekNonGeneric@inf.is>
@DerekNonGeneric
Copy link
Member Author

i am not seeing this when running locally, but in CI here is the error:

nps is executing compile.buildImages : tsx build/tasks/compile/build-images.mts
/home/runner/work/openinf.github.io/openinf.github.io/node_modules/.pnpm/sharp@0.33.5/node_modules/sharp/lib/input.js:487
const stack = Error();
^

Error: Input file contains unsupported image format
at Sharp.metadata (/home/runner/work/openinf.github.io/openinf.github.io/node_modules/.pnpm/sharp@0.33.5/node_modules/sharp/lib/input.js:487:17)
at Image.resize (/home/runner/work/openinf.github.io/openinf.github.io/node_modules/.pnpm/@11ty+eleventy-img@5.0.0/node_modules/@11ty/eleventy-img/img.js:545:37)
at /home/runner/work/openinf.github.io/openinf.github.io/node_modules/.pnpm/@11ty+eleventy-img@5.0.0/node_modules/@11ty/eleventy-img/img.js:870:16
at async run (/home/runner/work/openinf.github.io/openinf.github.io/node_modules/.pnpm/p-queue@6.6.2/node_modules/p-queue/dist/index.js:163:29)

Node.js v20.18.0
The script called "compile.buildImages" which runs "tsx build/tasks/compile/build-images.mts" failed with exit code 1 https://github.com/sezna/nps/blob/master/other/ERRORS_AND_WARNINGS.md#failed-with-exit-code

since there is only one, i think the way i'd like to solve this is by manually resizing the image and simply removing the @11ty/eleventy-img dependency (which i don't even really like)

@DerekNonGeneric DerekNonGeneric added the 🗳️ Status: Author Ready PRs that are considered ready by their author label Oct 27, 2024
@DerekNonGeneric DerekNonGeneric added the 🚅 Status: Fast Track PRs that do not need to wait for 48 hours to land label Oct 27, 2024
@DerekNonGeneric DerekNonGeneric merged commit 6280bf3 into OpenINF:live Oct 27, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🗞️ Category: Blog undefined 🏗️ Category: Build Issues and PRs related to the build directory ⚙️ Category: Engineering Problems/updates involving the functionality of the product 🏗️ Category: Infrastructure/Tooling/Builds/CI undefined 🟨 Code: JS/TS PRs or issues regarding JavaScript/TypeScript code 🗳️ Status: Author Ready PRs that are considered ready by their author 🚅 Status: Fast Track PRs that do not need to wait for 48 hours to land 📦 Type: Dependencies PRs that modify packages req'd by product in production 📦 Type: Developer Dependencies PRs that modify packages only req'd for local development and testing ✨ Type: Enhancement undefined 🈺 Type: Meta PRs or issues regarding the general management of the project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🏷️ migrating from jekyll to eleventy

2 participants