Skip to content

adding contributing guidance - #20

Merged
MathewBiddle merged 5 commits into
ioos:masterfrom
MathewBiddle:contribution
Sep 10, 2021
Merged

adding contributing guidance#20
MathewBiddle merged 5 commits into
ioos:masterfrom
MathewBiddle:contribution

Conversation

@MathewBiddle

Copy link
Copy Markdown
Contributor

@ocefpaf can you check for any inconsistencies?

Comment thread CONTRIBUTING.md Outdated
Comment thread CONTRIBUTING.md Outdated
Comment thread CONTRIBUTING.md Outdated
Co-authored-by: Filipe <ocefpaf@gmail.com>
Comment thread CONTRIBUTING.md Outdated

@ocefpaf ocefpaf left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Minor comments but LGTM!

@MathewBiddle

Copy link
Copy Markdown
Contributor Author

@ocefpaf, I made the change master -> main in the docs, but we should update the repository too. Not sure what the impacts might be (links and refs).

@MathewBiddle
MathewBiddle merged commit 2213628 into ioos:master Sep 10, 2021
@ocefpaf

ocefpaf commented Sep 13, 2021

Copy link
Copy Markdown
Member

@ocefpaf, I made the change master -> main in the docs, but we should update the repository too. Not sure what the impacts might be (links and refs).

Doing it now. We can fix the links as we go but you will need to update your fork and use main instead of master in the future.

@ocefpaf

ocefpaf commented Sep 13, 2021

Copy link
Copy Markdown
Member

Done! You will have to issue these commands locally on your fork/clone:

git branch -m master main
git fetch origin
git branch -u origin/main main
git remote set-head origin -a

You will also need to rename it in your fork to avoid confusion. GH should have the instructions on your fork when you navigate to it but let me know if you need any help navigating that.

@MathewBiddle

Copy link
Copy Markdown
Contributor Author

Thanks @ocefpaf, think I got it situated now.

@MathewBiddle
MathewBiddle deleted the contribution branch September 13, 2021 19:58
@MathewBiddle MathewBiddle linked an issue Sep 13, 2021 that may be closed by this pull request
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.

move master branch to main

2 participants