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

[CI] Docsy 0.8.0 NPM-package postinstall may fail to run #2116

Closed
chalin opened this issue Nov 6, 2024 · 1 comment · Fixed by #2117
Closed

[CI] Docsy 0.8.0 NPM-package postinstall may fail to run #2116

chalin opened this issue Nov 6, 2024 · 1 comment · Fixed by #2117
Assignees
Labels
CI/infra CI & infrastructure
Milestone

Comments

@chalin
Copy link
Collaborator

chalin commented Nov 6, 2024

This was uncovered by @sftim through his work on

To reproduce the problem, run the following:

hugo new site --format yaml myproject
cd myproject
echo 'theme: docsy' >> hugo.yaml
echo 'themesDir: node_modules' >> hugo.yaml
npm init -y
npm install --save-dev google/docsy#semver:0.8.0 autoprefixer postcss-cli hugo-extended@0.120.4
npx hugo 

Hugo reports: "Error: failed to load modules: module "github.com/FortAwesome/Font-Awesome" not found in "/Users/chalin/tmp/myproject/node_modules/github.com/FortAwesome/Font-Awesome"; either add it as a Hugo Module or store it in "/Users/chalin/tmp/myproject/node_modules".: module does not exist"

The problem is that Docsy's postinstall NPM script isn't being run automatically, but it should.

@chalin chalin added the CI/infra CI & infrastructure label Nov 6, 2024
@chalin chalin self-assigned this Nov 6, 2024
@chalin chalin changed the title [CI] Docsy 0.8.0 NPM-package use doesn't work out-of-the-box [CI] Docsy 0.8.0 NPM-package use may fail Nov 6, 2024
@sftim
Copy link

sftim commented Nov 6, 2024

Ah, thanks for identifying this.

@chalin chalin changed the title [CI] Docsy 0.8.0 NPM-package use may fail [CI] Docsy 0.8.0 NPM-package postinstall may fail to run Nov 12, 2024
@chalin chalin added this to the 24Q4 milestone Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/infra CI & infrastructure
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants