Skip to content

feat: deposit register.csv row to CODECHECK Register on publish (closes #10)#147

Merged
nuest merged 2 commits into
mainfrom
feature/10-register-deposit
Jul 13, 2026
Merged

feat: deposit register.csv row to CODECHECK Register on publish (closes #10)#147
nuest merged 2 commits into
mainfrom
feature/10-register-deposit

Conversation

@subhanLabs

Copy link
Copy Markdown
Member

This PR is built on top of #139 (repository-selection / repoWithCodecheckYaml) and should be merged after it.

Summary

When a CODECHECK-opted-in article is published, the plugin now automatically opens a PR against the CODECHECK Register adding a row for it, closing #10.

What this adds

  • CodecheckRegisterDepositService — resolves the repository the codechecker marked as containing codecheck.yml (from Feature/publication validation #139), re-validates it, formats it for the register (github::org/repo, zenodo::id, osf::id, gitlab::path), and builds the row.
  • CodecheckGithubRegisterApiClient::depositRegisterRow() — appends the row to register.csv and opens a PR against the configured register repo.
  • CodecheckPlugin::depositToRegister() — hooked into Publication::publish; runs for opted-in submissions; never blocks publication if the deposit fails.

Tested against

codecheckers/testing-dev-register, using codecheckers/driftage as an existing repo with a real codecheck.yml. Working example: codecheckers/testing-dev-register#176

How to test

  1. Set an opted-in submission's title to match an existing codecheck.yml in a codecheckers/* repo (or your own Zenodo/OSF record).
  2. Add that repository, check "Contains codecheck.yml file," reserve a certificate identifier.
  3. Publish, then check the logs and testing-dev-register's open PRs.
image image

@nuest

nuest commented Jul 13, 2026

Copy link
Copy Markdown
Member

@subhanLabs can you please rebase so that this PR only contains your changes? Thanks!

Resolves the codechecker-selected repository (#146), re-validates its
codecheck.yml, and opens a PR against the register with the new row.
Tested against testing-dev-register: codecheckers/testing-dev-register#176
@subhanLabs
subhanLabs force-pushed the feature/10-register-deposit branch from 80c5418 to 85d7e78 Compare July 13, 2026 11:36
@nuest
nuest merged commit dd0d7fe into main Jul 13, 2026
3 checks passed
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