-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add a codechecker view to the register #30
Comments
The |
happy to use Rmd for this, and hence gluedown. re: layout specifics, I'm not sure which is best (what is easiest for now, and if we can easily swap out one for another, does it matter what we choose?) |
Right now everything is plain Jekyll, so not using R Markdown is probably easier. Will see, this is not high on the list yet with just the two of us as checkers ;-) |
@nuest and @sje30 Couple of questions I need to ask about this feature:
If you prefer the former how would you like the string to be formatted? Some of the formats I could think of For the formats with the ORCID IDs I can add hyperlinks to the corresponding ORCID pages if you'd like.
I personally lean towards the ORCID ID atm. The codecheck.yaml files only contain the ORCID IDs of the codecheckers but not the GitHub handle. While it may be possible to look up GitHub handles for a specified name using GitHub API it may prove to be tricky for cases where using the full name does not return the correct GitHub handle or there are multiple users with the same full name. I would suggest creating a separate PR for adding the GitHub handles that I can look into later.
For the last format we could add the ORCID ID as a subtitle. |
Re. 1: Leave the Re. 2: ORCID, please. We would like to stay platform independent, who knows what Micro$oft might decide for GH at some point. Re. 3: |
Closing this issue as the basic feature for this has been merged |
The register could compile a codechecker profile page for each person contributing as a CODECHECKER.
https://codecheck.org.uk/register/codechecker/<GH handle>
(or betterORCID
?); content:stats.json
, add the number of codecheckers.The text was updated successfully, but these errors were encountered: