Skip to content

Setup RTD#295

Merged
atteggiani merged 4 commits intomainfrom
davide/setup_rtd
Nov 28, 2025
Merged

Setup RTD#295
atteggiani merged 4 commits intomainfrom
davide/setup_rtd

Conversation

@atteggiani
Copy link
Copy Markdown
Collaborator

@atteggiani atteggiani commented Nov 27, 2025

  • Updated deployment workflow
  • Added setup for RTD
  • Added documentation to paths-ignore

My admin permissions can be revoked now (please leave write permissions).

@atteggiani atteggiani self-assigned this Nov 27, 2025
@atteggiani atteggiani force-pushed the davide/setup_rtd branch 2 times, most recently from 6de142d to 9950701 Compare November 27, 2025 03:25
@atteggiani atteggiani force-pushed the davide/setup_rtd branch 3 times, most recently from 31edf61 to 3b5441c Compare November 27, 2025 03:47
@atteggiani atteggiani requested a review from blimlim November 27, 2025 03:55
blimlim
blimlim previously approved these changes Nov 28, 2025
Copy link
Copy Markdown
Collaborator

@blimlim blimlim left a comment

Choose a reason for hiding this comment

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

Thanks @atteggiani, I'm not 100% across the details, but have double checked the changes against the OM3 repo, and it all looks ok to me! Just noticed one tiny comment which could be updated

Comment thread .github/workflows/deploy_to_github_pages.yml Outdated
Co-authored-by: Spencer Wong <88933912+blimlim@users.noreply.github.com>
Copy link
Copy Markdown
Collaborator

@blimlim blimlim left a comment

Choose a reason for hiding this comment

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

Looks good!

@atteggiani atteggiani merged commit a4935d6 into main Nov 28, 2025
27 checks passed
@atteggiani atteggiani deleted the davide/setup_rtd branch November 28, 2025 04:53
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