Skip to content

[QA-250] Update QA metadata checks - #94

Draft
carly-jones wants to merge 11 commits into
developfrom
QA-250-update-metadata-check-reference
Draft

[QA-250] Update QA metadata checks#94
carly-jones wants to merge 11 commits into
developfrom
QA-250-update-metadata-check-reference

Conversation

@carly-jones

@carly-jones carly-jones commented Jul 24, 2026

Copy link
Copy Markdown

This PR replaces all imports of arxiv.metadata with imports from the updated qa package.

This PR also lightly refactors to make QA-related unit tests less fragile. This is desired so that changes to a QA check's logic or on_failure_policy will not break tests here.

Notes

  • Submit CE currently blocks when a check result has either a WARN or REJECT disposition. This PR updates this to only block on REJECT, as currently desired by EUST.
  • This PR does not display WARN messages to the submitter, although this can easily be added in the future.
  • This PR does not update submit-ce's pin of arxiv-base, but instead adds an override so that any submit-ce deps (including qa) which require arxiv-base use the version declared by submit-ce. This is currently a pretty stale commit but that's ok for qa - we only use a single helper that exists at that commit.

@carly-jones

Copy link
Copy Markdown
Author

Currently in draft until qa changes are merged to master in arxiv-base (will update the pin here)

@carly-jones
carly-jones marked this pull request as ready for review July 27, 2026 19:08
@carly-jones

Copy link
Copy Markdown
Author

Pin updated, ready for review

@carly-jones carly-jones changed the title [QA-250] Update QA checks [QA-250] Update QA metadata checks Jul 28, 2026
@carly-jones

Copy link
Copy Markdown
Author

Putting back in draft as I update checks in qa

@carly-jones
carly-jones marked this pull request as draft July 28, 2026 20:33
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.

1 participant