Skip to content

Conversation

@Cadair
Copy link
Member

@Cadair Cadair commented Jul 16, 2024

There's more to come, but I thought I would start collecting info. Only problem is I don't have everyone's GitHub handles.

Also I have run out of time for this today.

@hayesla
Copy link
Member

hayesla commented Nov 27, 2024

whats the status of this? i can take over and finish this PR if you want @Cadair

@Cadair
Copy link
Member Author

Cadair commented Nov 27, 2024

Please do :)

@hayesla

This comment was marked as outdated.

@nabobalis

This comment was marked as outdated.

@nabobalis nabobalis marked this pull request as ready for review March 3, 2025 23:07
@Cadair
Copy link
Member Author

Cadair commented Mar 26, 2025

Various style improvements and fixes are here: sunpy/sunpy-sphinx-theme#283 but that doesn't need to block this PR.

I think this is ready for final review.

Copy link
Member

@hayesla hayesla left a comment

Choose a reason for hiding this comment

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

I think its good to go, we can fix the smaller style things in another PR

@Cadair Cadair merged commit 58612be into sunpy:main Mar 26, 2025
4 of 5 checks passed
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.

4 participants