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

Fix broken urls in README #2408

Merged
merged 2 commits into from
Oct 26, 2023

Conversation

aknierim
Copy link
Contributor

This fixes broken urls in the README after the restructuring of the docs.

@kosack kosack added the documentation-only Label that will ensure code tests are skipped label Oct 11, 2023
Tobychev
Tobychev previously approved these changes Oct 16, 2023
@maxnoe
Copy link
Member

maxnoe commented Oct 22, 2023

Rebasing with main will fix the docs build

LukasNickel
LukasNickel previously approved these changes Oct 25, 2023
@aknierim aknierim dismissed stale reviews from LukasNickel and Tobychev via 284440f October 25, 2023 07:48
@aknierim aknierim requested a review from HealthyPear as a code owner October 25, 2023 07:48
@maxnoe
Copy link
Member

maxnoe commented Oct 25, 2023

Something went wrong in your rebasing

@aknierim
Copy link
Contributor Author

Something went wrong in your rebasing

Yeah, can't figure out what, though...

@maxnoe
Copy link
Member

maxnoe commented Oct 25, 2023

Did you update upstream before the rebase?

git fetch upstream
git rebase upstream/main

?

@aknierim
Copy link
Contributor Author

Yup, which is why I'm confused

Set navigation_with_keys to False to get rid of the
warning. Setting it to True has negative accessibility
implications: pydata/pydata-sphinx-theme#1492
@maxnoe maxnoe force-pushed the fix_broken_urls_in_readme branch from e4f3a9c to 38d38d8 Compare October 25, 2023 09:39
@maxnoe maxnoe merged commit 273f642 into cta-observatory:main Oct 26, 2023
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation-only Label that will ensure code tests are skipped no-changelog-needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants