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

the funder registry moved over to gitlab #358

Merged
merged 1 commit into from
May 8, 2020
Merged

Conversation

kf8a
Copy link
Contributor

@kf8a kf8a commented May 5, 2020

Just a very small documentation fix adjusting the url to the funder registry

@amoeba amoeba changed the base branch from master to develop May 8, 2020 20:10
@amoeba
Copy link
Contributor

amoeba commented May 8, 2020

Looks good to me, thanks for seeing the old link and fixing it for us @kf8a.

@mbjones I wasn't sure which branch to merge this onto so I'll leave this unmerged for now. Our docs indicate new commits go onto the latest development branch (e.g., BRANCH_EML_2_2) but I see you are also working over on develop. Where should we be adding commits?

@mbjones
Copy link
Member

mbjones commented May 8, 2020

Yes, we haven't been as explicit as we should with a CONTRIBUTING.md file. The BRANCH_EML_2_2 is really for schema-related changes on the 2.2 series of releases. So, I created a develop branch to be used as a general integration/testing/documentation branch and have been making most changes there that don't relate to a schema release per se. Then that gets merged into master. So, please do merge into develop, and note that there are other pending changes there awaiting review in issue #355 (but I think it is ready to go as described).

@amoeba
Copy link
Contributor

amoeba commented May 8, 2020

Sounds great, thanks.

@amoeba amoeba merged commit bb9fc0a into NCEAS:develop May 8, 2020
@mbjones mbjones added this to the EML3.0.0 milestone Feb 23, 2021
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