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] Smoke test Docsy-based site using Hugo-modules #1794

Closed
Tracked by #1780
chalin opened this issue Jan 19, 2024 · 1 comment · Fixed by #1795
Closed
Tracked by #1780

[CI] Smoke test Docsy-based site using Hugo-modules #1794

chalin opened this issue Jan 19, 2024 · 1 comment · Fixed by #1795
Labels
CI/infra CI & infrastructure e0-minutes Effort < 60 min p2-medium p3-low
Milestone

Comments

@chalin
Copy link
Collaborator

chalin commented Jan 19, 2024

Contributes to:

It may be necessary to only run these smoke tests over PRs/branches in this repo. I've tried fetching from a branch in my fork, for example, but Go complains (rightly) that modules paths aren't as expected:

$ hugo mod get github.com/chalin/docsy@chalin-im-blog-post-lastmod-2024-01-15
go: downloading github.com/chalin/docsy v0.0.0-20240115234354-346c42b292cc
go: github.com/chalin/docsy@chalin-im-blog-post-lastmod-2024-01-15 (v0.0.0-20240115234354-346c42b292cc) requires github.com/chalin/docsy@v0.0.0-20240115234354-346c42b292cc: parsing go.mod:
        module declares its path as: github.com/google/docsy
                but was required as: github.com/chalin/docsy

And I can't seem to specify a replace rule from the command line.

@chalin chalin added e0-minutes Effort < 60 min CI/infra CI & infrastructure p2-medium p3-low labels Jan 19, 2024
@chalin chalin added this to the 24Q1 milestone Jan 19, 2024
@chalin
Copy link
Collaborator Author

chalin commented Jan 20, 2024

I've found a way to work from PR branches.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/infra CI & infrastructure e0-minutes Effort < 60 min p2-medium p3-low
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant