Skip to content

Codemeta v4 daniel#12

Open
dgarijo wants to merge 38 commits into
target_codemeta_v4from
codemeta_v4_daniel
Open

Codemeta v4 daniel#12
dgarijo wants to merge 38 commits into
target_codemeta_v4from
codemeta_v4_daniel

Conversation

@dgarijo

@dgarijo dgarijo commented May 19, 2026

Copy link
Copy Markdown

No description provided.

Aidajafarbigloo and others added 22 commits October 29, 2025 11:11
Co-authored-by: Val Lorentz <progval+github@progval.net>
Keeping only `http_url_to_repo `. In response to codemeta#444 (comment) and codemeta#444 (comment).
Removed `_links` in response to codemeta#444 (comment).
@dgarijo

dgarijo commented May 19, 2026

Copy link
Copy Markdown
Author

Not yet ready to merge

@dgarijo dgarijo marked this pull request as draft May 19, 2026 13:01
@dgarijo dgarijo marked this pull request as ready for review June 23, 2026 16:44
@dgarijo

dgarijo commented Jun 23, 2026

Copy link
Copy Markdown
Author

Ready to merge

@lindangulopez lindangulopez left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

My name should be Linda Angulo Lopez

Comment thread codemeta_all.jsonld Outdated
Comment thread codemeta.jsonld Outdated

@lindangulopez lindangulopez left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

just a TODO after v4: for our documentation.

Comment thread CONTRIBUTORS.MD
| Maxence Azzouz| Fiz Karlsruhe University | [Mazztok45](https://github.com/Mazztok45) |
| Allen Lee | ASU/OMF | [allenlee](https://github.com/allenlee) |
| Jael García Castro| ZBMED | [ljgarcia](https://github.com/ljgarcia) |
| Jozefina Sadowska | Inria | []() |

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

missing @ as is @lindangulopez in line 137

@lindangulopez lindangulopez left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

missing @ for @lindangulopez and for Jozefina Sadowska

@progval progval left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Thanks.

Could you open separate PRs with the changes to GitLab.csv, aggregate.py, and split.py?

Then we'll merge them and squash this PR into a single commit.

@dgarijo

dgarijo commented Jun 26, 2026 via email

Copy link
Copy Markdown
Author

@progval

progval commented Jun 26, 2026

Copy link
Copy Markdown

Oh sorry, Linda linked me this PR and I didn't realize it didn't target the main repo. Alright then

@lindangulopez

Copy link
Copy Markdown

Thanks for reviewing and for clarifying!

@dgarijo

dgarijo commented Jun 26, 2026

Copy link
Copy Markdown
Author

No worries, I had the same concern when I updated. I will do the pending changes and do a PR in CodeMeta, I think it's just a couple of things missing regarding 2 new properties.

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.

7 participants