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

Registers package with sunpy template #120

Merged
merged 24 commits into from
Oct 16, 2024

Conversation

CyclingNinja
Copy link
Member

No description provided.

.black Show resolved Hide resolved
.github/workflows/ci.yml Show resolved Hide resolved
docs/conf.py Outdated Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
tox.ini Outdated Show resolved Hide resolved
tox.ini Outdated Show resolved Hide resolved
tox.ini Show resolved Hide resolved
tox.ini Outdated Show resolved Hide resolved
pyproject.toml Show resolved Hide resolved
pytest.ini Show resolved Hide resolved
pytest.ini Show resolved Hide resolved
CyclingNinja and others added 3 commits September 12, 2024 18:30
Co-authored-by: Stuart Mumford <stuart@cadair.com>
Co-authored-by: Stuart Mumford <stuart@cadair.com>
.black Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
docs/index.rst Outdated Show resolved Hide resolved
@CyclingNinja CyclingNinja force-pushed the register_template branch 2 times, most recently from 4446c09 to 2a2176d Compare October 15, 2024 14:43
@Cadair
Copy link
Member

Cadair commented Oct 15, 2024

pre-commit.ci autofix

@Cadair
Copy link
Member

Cadair commented Oct 16, 2024

@nabobalis The online build is failing with some errors in tutorial.rst and I am not sure if they are real errors or just weirdness.

@nabobalis
Copy link
Contributor

That looks like a real error but I can't imagine it comes from this PR

@Cadair
Copy link
Member

Cadair commented Oct 16, 2024

:shipit: ?

@nabobalis
Copy link
Contributor

Also the CI has not run for like 3 months, so it's possible it's just broken and we never knew.

@nabobalis
Copy link
Contributor

I am not maintaining DRMS, so I leave the review and merge to someone who is.

@Cadair Cadair merged commit e95f68a into sunpy:main Oct 16, 2024
14 of 15 checks passed
@Cadair
Copy link
Member

Cadair commented Oct 16, 2024

Well in that case, executive power deployed.

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.

3 participants