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

🗃️ Include review metadata #35

Open
shnizzedy opened this issue Feb 3, 2024 · 1 comment
Open

🗃️ Include review metadata #35

shnizzedy opened this issue Feb 3, 2024 · 1 comment

Comments

@shnizzedy
Copy link
Member

[W]e want to include a timestamp from when the review was completed and some record of who completed the review. These can be separate PRs, especially since I don't think we ever decided how we want to authenticate / credit reviewers.

#30 (review)

@nx10
Copy link
Contributor

nx10 commented Feb 14, 2024

The data structure used in https://github.com/nmind/proceedings has deviated a bit from the new schema. I wrote a script to migrate the new format, but the following fields do not currently have an equivalent in the checklist schema:

field type clarification
description string tool description
image string path to tool logo/icon (could be a URL)
toolVersion string tool version
date string date of review
maintainers obj[] array of name+email+affiliation of tool maintainers
evaluatorEmail string email address of evaluator

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 a pull request may close this issue.

2 participants