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

Remove previous JWS validation and structs #34

Merged
merged 1 commit into from
Oct 8, 2024
Merged

Remove previous JWS validation and structs #34

merged 1 commit into from
Oct 8, 2024

Conversation

theory
Copy link
Member

@theory theory commented Oct 8, 2024

With the switch from the old release field to the new certs and release fields as defined in RFC 5 and implemented in 0773ad7 and acb0d22, the previously-defined JSON schemas and data structure are no longer used. So remove them and their tests.

With the switch from the old `release` field to the new `certs` and
`release` fields as defined in [RFC 5] and implemented in 0773ad7 and
acb0d22, the previously-defined JSON schemas and data structure are no
longer used. So remove them and their tests.

  [RFC 5]: pgxn/rfcs#5
@theory theory requested a review from vrmiguel October 8, 2024 16:24
@theory theory self-assigned this Oct 8, 2024
Copy link

codecov bot commented Oct 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (acb0d22) to head (156d314).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #34   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           10        10           
  Lines         1131      1105   -26     
=========================================
- Hits          1131      1105   -26     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Base automatically changed from release-certs to main October 8, 2024 19:43
@theory theory merged commit 156d314 into main Oct 8, 2024
138 checks passed
@theory theory deleted the certs-cleanup branch October 8, 2024 21:12
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