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

[editorial] Anchor IDs do not exist in v1.27 of docs #1313

Closed
2 tasks done
chalin opened this issue Aug 2, 2024 · 3 comments · Fixed by #1354
Closed
2 tasks done

[editorial] Anchor IDs do not exist in v1.27 of docs #1313

chalin opened this issue Aug 2, 2024 · 3 comments · Fixed by #1354
Assignees

Comments

@chalin
Copy link
Contributor

chalin commented Aug 2, 2024


These issues were reported by the link checker via the GitHub action checks of:

In particular, see the "hash does not exist" errors reported in: https://github.com/open-telemetry/opentelemetry.io/actions/runs/10218743437/job/28275657390?pr=4938

@lmolkova
Copy link
Contributor

lmolkova commented Aug 2, 2024

Thanks @chalin !

Is there a way we can easily test it before releasing? I.e. can we somehow trigger docs pipeline for a specific commit in the semconv repo?

We used to have cross-file link checker, but it's gone and we have no replacement for it across all repos - #1009.

@chalin
Copy link
Contributor Author

chalin commented Aug 2, 2024

The easiest way that I can think of, would be to build the OTel website using the pre-release commit of this repo. We could define a manually triggered workflow that could build the OTel website using main@HEAD, which could be run just before putting out the release. /cc @open-telemetry/docs-maintainers

I suppose that the same workflow could be run via a cron job over main@HEAD, but that might report more errors than it's worth? Maybe not.

@chalin
Copy link
Contributor Author

chalin commented Aug 2, 2024

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.

3 participants