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

added initial mappings for cargo.toml (rust package manager) #111

Merged
merged 2 commits into from
Nov 23, 2018

Conversation

encima
Copy link
Contributor

@encima encima commented Sep 15, 2016

I believe crates.io adds its own metadata but I have only added tags that pertain to the mainfest structure of a cargo.toml file.

I also could not see a specific key for documentation (I know there is DocsCoverage), but that is for a separate discussion and, thus, I have left out the documentation link.

@cboettig
Copy link
Member

Can't merge multiple PR's to a csv (wouldn't be an issue if the crosswalk was in JSON...)

This conflicts with the just-merged PR #170. Can you take a quick go at resolving that (working from the mobile here so I won't attempt it right now)

@cboettig
Copy link
Member

p.s. can you also add yourself to the contributors under https://github.com/codemeta/codemeta/blob/master/codemeta.json?

@cboettig
Copy link
Member

Thanks for the PR. because having multiple PRs against a shared CV creates a lot of merge conflicts, we've broken the crosswalk table into separate files that are now merged automatically. Would you be able to resubmit this with this addition as a separate .csv file, as you see now in the master branch for the other entries?

Thanks!

@encima
Copy link
Contributor Author

encima commented Nov 19, 2018 via email

@cboettig
Copy link
Member

👍

@cboettig cboettig merged commit 0ba1cbd into codemeta:master Nov 23, 2018
proycon pushed a commit to proycon/codemeta that referenced this pull request Jan 4, 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.

None yet

2 participants