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

Standardize frontmatter across Tutorials #29

Merged
merged 12 commits into from
Dec 5, 2024

Conversation

yvonnefroehlich
Copy link
Member

@yvonnefroehlich yvonnefroehlich commented Dec 5, 2024

Related to #1 (comment), this PR aims to standardize the frontmatter across the tutorials:

  • Mention pre-workshop (Jupyter book, GitHub, AGU Session)
  • History (author, creation date, recommended versions)
  • Consistent titles, also with intro page (if you don't like to have emojis in your tutorial, just comment on this 🙂)
  • Links to the PyGMT documentation
  • Mention the GitHub repo for errors and the GMT forum for questions
  • Make headings consistent - Please check for your tutorial if that the structure of headings is correct!

@yvonnefroehlich yvonnefroehlich added the documentation Improvements or additions to documentation label Dec 5, 2024
@yvonnefroehlich yvonnefroehlich self-assigned this Dec 5, 2024
Copy link

gitnotebooks bot commented Dec 5, 2024

Found 5 changed notebooks. Review the changes at https://app.gitnotebooks.com/GenericMappingTools/agu24workshop/pull/29

@yvonnefroehlich yvonnefroehlich changed the title Standarzise frontmatter across Tutorials Standardize frontmatter across Tutorials Dec 5, 2024
@yvonnefroehlich yvonnefroehlich added needs review Review of this PR is welcome maintenance labels Dec 5, 2024
Copy link
Collaborator

@andrebelem andrebelem left a comment

Choose a reason for hiding this comment

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

Perfect ! All good.

Copy link
Member

@maxrjones maxrjones left a comment

Choose a reason for hiding this comment

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

This is great! I pushed a couple small changes rather than providing a review given the limited time remaining. Thank you for these improvements!

@maxrjones
Copy link
Member

@yvonnefroehlich are you planning to make any more changes to this? It'd be great to merge if not

@yvonnefroehlich
Copy link
Member Author

This is great! I pushed a couple small changes rather than providing a review given the limited time remaining. Thank you for these improvements!

No worries! This is totally fine. And thanks for doing the rst related stuff 🙂.

@yvonnefroehlich are you planning to make any more changes to this? It'd be great to merge if not

At the moment I will not work on this. So, feel free to merge this PR. Then people can continue with other PRs (code linting etc.).

@maxrjones maxrjones merged commit 45705b7 into main Dec 5, 2024
1 check passed
@maxrjones maxrjones deleted the make-frontmatter-consistent branch December 5, 2024 16:17
@yvonnefroehlich yvonnefroehlich removed the needs review Review of this PR is welcome label Dec 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants