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.

Sign up for GitHub

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

Jump to bottom

ARXIVCE-825 Metadata checks: first commit #352

Merged

jonathanhyoung merged 11 commits into develop from ARXIVCE-825

Feb 25, 2025

Contributor

jonathanhyoung commented Feb 11, 2025

No description provided.


          ARXIVCE-825 Metadata checks: first commit

59cc613

jonathanhyoung requested review from jweiskoff and a team

February 11, 2025 20:16

jweiskoff approved these changes

View reviewed changes

Jonathan Young added 2 commits

February 11, 2025 16:34


          ARXIVCE-825 add wheel to fix build.

801774a


          ARXIVCE-825 add wheel to fix build.

189a2ab

bdc34 self-requested a review

February 11, 2025 22:16

bdc34 requested changes

View reviewed changes

Contributor

bdc34 left a comment

Due to being used by many other repos, arxiv-base must pass it's tests.

Jonathan Young added 2 commits

February 14, 2025 12:39


          Install protoc for gcld3.

5ebb6ea


          Install protoc for gcld3 using sudo.

bd1efa1

jonathanhyoung requested a review from bdc34

February 14, 2025 18:04

bdc34 reviewed

View reviewed changes

arxiv/metadata/metacheck.py Outdated Show resolved Hide resolved

bdc34 reviewed

View reviewed changes

arxiv/metadata/metacheck.py Outdated Show resolved Hide resolved

bdc34 reviewed

View reviewed changes

arxiv/metadata/metacheck.py Show resolved Hide resolved

bdc34 reviewed

View reviewed changes

arxiv/metadata/metacheck.py Outdated Show resolved Hide resolved

bdc34 reviewed

View reviewed changes

arxiv/metadata/metacheck.py Show resolved Hide resolved

bdc34 reviewed

View reviewed changes

arxiv/metadata/metacheck.py Show resolved Hide resolved

bdc34 reviewed

View reviewed changes

arxiv/metadata/metacheck.py Show resolved Hide resolved

bdc34 reviewed

View reviewed changes

arxiv/metadata/metacheck.py Outdated Show resolved Hide resolved

bdc34 reviewed

View reviewed changes

arxiv/metadata/metacheck.py Outdated Show resolved Hide resolved

bdc34 reviewed

View reviewed changes

arxiv/metadata/metacheck.py Outdated Show resolved Hide resolved

bdc34 reviewed

View reviewed changes

arxiv/metadata/metacheck.py Outdated Show resolved Hide resolved

bdc34 reviewed

View reviewed changes

arxiv/metadata/metacheck.py Outdated Show resolved Hide resolved

bdc34 reviewed

View reviewed changes

arxiv/metadata/metacheck.py Outdated Show resolved Hide resolved

bdc34 reviewed

View reviewed changes

arxiv/metadata/metacheck.py Outdated Show resolved Hide resolved

bdc34 reviewed

View reviewed changes

arxiv/metadata/metacheck.py Show resolved Hide resolved

bdc34 reviewed

View reviewed changes

arxiv/metadata/tests/test_metacheck.py Outdated Show resolved Hide resolved

bdc34 requested changes

View reviewed changes

Contributor

bdc34 left a comment

looks good. One requested change where a complaint isn't the correct one.
I'd consider splitting metacheck.py out into author_checks.py abstract_checks.py etc for easier examination of when the checks changed using just a single file git log.

Jonathan Young added 2 commits

February 21, 2025 14:50


          Merge branch 'develop' into ARXIVCE-825

9a150a8


          ARXIVCE-825 Changes from code review.

592b500

Jonathan Young added 4 commits

February 24, 2025 14:43


          ARXIVCE-825 feedback from code review.

d9975ad


          ARXIVCE-825 more feedback from code review.

be6f8c1


          ARXIVCE-825 more feedback from code review.

b1f8b7b


          ARXIVCE-825 more feedback from code review.

6fccaec

jonathanhyoung requested a review from bdc34

February 25, 2025 01:45

bdc34 approved these changes

View reviewed changes

Contributor

bdc34 left a comment

Thanks!

jonathanhyoung merged commit 4e405cc into develop

1 check passed

jonathanhyoung deleted the ARXIVCE-825 branch

February 25, 2025 20:46

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet