-
Notifications
You must be signed in to change notification settings - Fork 8.1k
upgrade build and dev dependencies #19275
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
Conversation
Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
Fetching the prebuilt hugo binary is faster than `go install`. Also switching to the extended version. Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
Separate dev dependencies like prettier to prevent them from being installed to build the site, unnecessarily. Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
We're not using mermaid for diagrams anymore so we can prune the dependency. Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
Signed-off-by: David Karlsson <35727626+dvdksn@users.noreply.github.com>
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
@crazy-max you don't have to bother with the nodejs stuff in this PR but just wanted to get your eyes on a1102d1 Moving off go install for Hugo and fetching a binary with wget instead. Also needs gcompat layer for building with this one. |
Yes looks good! |
Proposed changes
Related issues (optional)