Skip to content

Conversation

@freemansw1
Copy link
Member

Updates the zenodo file to fix the bugs. I used this website: https://www.jsonschemavalidator.net/ to validate the JSON file and then used this schema description: https://zenodraft.github.io/metadata-schema-zenodo/latest/schema.json .

@freemansw1 freemansw1 requested a review from ldgrantcsu July 20, 2022 17:30
Copy link
Member

@ldgrantcsu ldgrantcsu left a comment

Choose a reason for hiding this comment

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

The zenodo file looks good to me. My only question @freemansw1 is why multiple previous commits you made are also showing up in the pull request. Is this from updating your fork?

@freemansw1
Copy link
Member Author

Yes, that's from merging my fork (and from us doing a squash merge earlier). We can squash merge to get rid of them or just deal with the commit history

@freemansw1
Copy link
Member Author

I should also note that updating my fork was a huge headache with the change in the folder inside of src. I would prefer that we get rid of that.

@ldgrantcsu
Copy link
Member

Agreed.. yes that causes some unnecessary extra steps. OK looks good to me.

Copy link
Member

@ldgrantcsu ldgrantcsu left a comment

Choose a reason for hiding this comment

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

Changes look good.

@freemansw1 freemansw1 merged commit db411ca into RAMSmodel:main Jul 20, 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.

2 participants