Skip to content

Add topic: argo-floats#2962

Merged
MikeMcQuaid merged 3 commits intogithub:mainfrom
euroargodev:main
Mar 3, 2022
Merged

Add topic: argo-floats#2962
MikeMcQuaid merged 3 commits intogithub:mainfrom
euroargodev:main

Conversation

@gmaze
Copy link
Copy Markdown
Contributor

@gmaze gmaze commented Mar 2, 2022

Please confirm this pull request meets the following requirements:

Which change are you proposing?

  • Suggesting edits to an existing topic or collection
  • Curating a new topic or collection
  • Something that does not neatly fit into the binary options above

Curating a new topic or collection

  • I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
  • My folder contains a *.png image (if applicable) and index.md
  • All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

The world climate science community is using more and more Github as a platform to develop and share codes related to Earth Observing Systems and in particular Argo floats. Adding a new topic 'argo-floats' would provide important information to this growing GitHub's community, by crucially helping code discovery through topic labelling and search in repositories.

@gmaze gmaze requested a review from a team as a code owner March 2, 2022 14:37
@PechSophea
Copy link
Copy Markdown

PechSophea commented Mar 2, 2022 via email

Copy link
Copy Markdown
Contributor

@MikeMcQuaid MikeMcQuaid left a comment

Choose a reason for hiding this comment

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

@gmaze
Copy link
Copy Markdown
Contributor Author

gmaze commented Mar 3, 2022

@gmaze Thanks for the PR! A couple of CI failures to address here: https://github.com/github/explore/runs/5395271023?check_suite_focus=true#step:4:17

All 3 errors fixed, thanks @MikeMcQuaid

Copy link
Copy Markdown
Contributor

@MikeMcQuaid MikeMcQuaid left a comment

Choose a reason for hiding this comment

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

Thanks so much for your first contribution! Without people like you submitting PRs we couldn't run this project. You rock, @gmaze!

@MikeMcQuaid MikeMcQuaid merged commit e4e07a2 into github:main Mar 3, 2022
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