Skip to content

Small tweaks to v3 - #99

Merged
Cadair merged 2 commits into
astropy:mainfrom
Cadair:tweaks_v3
May 11, 2026
Merged

Small tweaks to v3#99
Cadair merged 2 commits into
astropy:mainfrom
Cadair:tweaks_v3

Conversation

@Cadair

@Cadair Cadair commented May 11, 2026

Copy link
Copy Markdown
Member

No description provided.

@Cadair
Cadair requested a review from pllim May 11, 2026 14:46
Comment thread README.rst
Comment thread README.rst Outdated
Comment thread README.rst Outdated
Comment thread sphinx_astropy/conf/v2.py
'matplotlib': ('https://matplotlib.org/stable/',
(None, 'http://data.astropy.org/intersphinx/matplotlib.inv')),
'astropy': ('https://docs.astropy.org/en/stable/', None),
'astropy.org': ('https://astropy.org/', None),

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we actually do this now? Is there a working example?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

yes it works since we merged the astropy.org PR. astropy/astropy#19206 is testing it on CI currently, I've tested it locally.

@Cadair
Cadair requested a review from pllim May 11, 2026 14:59
Comment thread README.rst Outdated
Comment thread README.rst Outdated
Comment thread README.rst Outdated
@Cadair

Cadair commented May 11, 2026

Copy link
Copy Markdown
Member Author

I cleaned up the history, I think we are good?

@pllim pllim left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Merge when you ready. Thanks!

@Cadair
Cadair merged commit 66e6ba9 into astropy:main May 11, 2026
16 checks passed
@Cadair
Cadair deleted the tweaks_v3 branch May 11, 2026 15:11
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