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

[WIP] Broken notebooks #605

Merged
merged 9 commits into from
Sep 27, 2024
Merged

[WIP] Broken notebooks #605

merged 9 commits into from
Sep 27, 2024

Conversation

jeffjennings
Copy link
Contributor

Fixes notebook builds that are failing due to various errors.

  • Check the box to confirm that you are familiar with the contributing guidelines and/or indicate (check the box) that you are familiar with our contributing workflow.
  • Confirm that any contributed tutorials contain a complete Introduction which includes an Author list, Learning Goals, Keywords, Companion Content (if applicable), and a Summary.
  • Check the box to confirm that you are familiar with the Astropy community code of conduct and you agree to follow the CoC.

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@jeffjennings jeffjennings marked this pull request as draft September 26, 2024 12:30
@jeffjennings
Copy link
Contributor Author

@adrn Can you add the 'Run all tutorials' label to this PR and #606? I have not the power.

@adrn adrn added the Run all tutorials Tells the GitHub actions CI to run all tutorials in the repo (vs. only modified ones) label Sep 26, 2024
@jeffjennings jeffjennings marked this pull request as ready for review September 26, 2024 14:21
@adrn adrn merged commit bc659a3 into astropy:main Sep 27, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Run all tutorials Tells the GitHub actions CI to run all tutorials in the repo (vs. only modified ones)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants