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

Update Netlify build command #167

Closed
chalin opened this issue Apr 4, 2022 · 2 comments · Fixed by #169
Closed

Update Netlify build command #167

chalin opened this issue Apr 4, 2022 · 2 comments · Fixed by #169

Comments

@chalin
Copy link
Collaborator

chalin commented Apr 4, 2022

For context, see #156 (comment):

Studying the deploy log I realized, that git submodules still get checked out. This doesn't do any harm, but it is not necessary either. Maybe this can be corrected? If so, I think we should open a separate ticket for this.

From #156 (comment):

OK, so the build instruction on Netlify is currently:

cd themes/docsy && git submodule update -f --init && cd ../.. && hugo

So will automatically check out the submodules. With Hugo Modules will it just work by running hugo?

/cc @deining @LisaFC

@chalin
Copy link
Collaborator Author

chalin commented Apr 4, 2022

@deining - I've updated the build command to be just hugo, and the build is failing with the following output:

11:26:53 AM: $ hugo
11:26:55 AM: hugo: downloading modules …
11:27:05 AM: go: github.com/google/docsy/dependencies@v0.2.0-pre.0.20220321183617-02df04c0f2d4: invalid pseudo-version: tag (v0.2.0-pre) found on revision 02df04c0f2d4 is already canonical, so should not be replaced with a pseudo-version derived from that tag
11:27:05 AM: hugo: collected modules in 12403 ms
11:27:05 AM: Error: failed to download modules: failed to execute 'go [mod download]': failed to execute binary "go" with args [mod download]: go: github.com/google/docsy/dependencies@v0.2.0-pre.0.20220321183617-02df04c0f2d4: invalid pseudo-version: tag (v0.2.0-pre) found on revision 02df04c0f2d4 is already canonical, so should not be replaced with a pseudo-version derived from that tag
11:27:05 AM:  *errors.errorString

Can you look into it? Thanks.

/cc @LisaFC

@deining
Copy link
Collaborator

deining commented Apr 4, 2022

Can you look into it? Thanks.

This issue can be closed via #169.

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 a pull request may close this issue.

2 participants