Skip to content

Fix Github pages relative paths - #430

Merged
awoods merged 3 commits into
mainfrom
fix_rel_paths
Aug 5, 2026
Merged

Fix Github pages relative paths#430
awoods merged 3 commits into
mainfrom
fix_rel_paths

Conversation

@sprater

@sprater sprater commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Fix relative paths for Github pages documents


What does this Pull Request do?

Modify the relative paths to resources under docs (Github pages) to remove /fits from the beginning of the path, as the fitstool.org domain now points to the fits repo Github pages as its root.

How should this be tested?

  1. Clone the repo in an environment that also has a GUI with a browser.
  2. Checkout the branch "fix_rel_paths".
  3. Change to the docs/ directory.
  4. If necessary, run bundle install to install all the jekyll dependencies. If running Ruby 3.4+ also run bundle add bigdecimal.
  5. Launch the test webserver: bundle exec jekyll serve
  6. In your browser,go to http://127.0.0.1:4000/ (as shown in the output of the bundle exec jekyll serve command)
  7. Confirm that the github pages display properly, and that the links work.

Interested parties

@awoods @aurora-charlow @ebenenglish

@awoods

awoods commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

@awoods awoods left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See general comment on PR

@sprater

sprater commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator Author

@awoods Good catch, that one escaped my regex. I fixed it, and I made sure there were no more links with that markdown syntax.
c367e53

@awoods
awoods merged commit 8613bf6 into main Aug 5, 2026
1 check failed
@awoods
awoods deleted the fix_rel_paths branch August 5, 2026 22:36
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 this pull request may close these issues.

2 participants