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

(PE-37345) Create new bulk signing endpoints #2799

Merged
merged 1 commit into from
Dec 21, 2023

Conversation

artlawson
Copy link
Contributor

No description provided.

@artlawson artlawson force-pushed the PE-37345 branch 14 times, most recently from c67a97a to a6f079c Compare December 18, 2023 22:05
@artlawson artlawson force-pushed the PE-37345 branch 3 times, most recently from b706d98 to 533ac24 Compare December 18, 2023 23:19
@artlawson artlawson force-pushed the PE-37345 branch 5 times, most recently from c6d2c55 to db30c07 Compare December 19, 2023 01:51
@artlawson artlawson marked this pull request as ready for review December 19, 2023 02:07
@artlawson artlawson requested a review from a team as a code owner December 19, 2023 02:07
@jonathannewman
Copy link
Contributor

This looks great. Would you mind adding a test that demonstrates the behavior when the payload doesn't comply with the schema. The tests can act as a point of documentation in this regard as well as a guarantee that we maintain consistent behavior.

@artlawson artlawson force-pushed the PE-37345 branch 2 times, most recently from f62ae3e to f96d9b4 Compare December 19, 2023 20:26
@artlawson artlawson force-pushed the PE-37345 branch 23 times, most recently from 63d5529 to 5af3792 Compare December 20, 2023 19:49
@artlawson artlawson force-pushed the PE-37345 branch 2 times, most recently from c0cfe90 to cd3bcad Compare December 20, 2023 23:52
Copy link
Contributor

@steveax steveax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice!
LGTM!

@steveax steveax merged commit f9971f4 into puppetlabs:main Dec 21, 2023
11 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.

3 participants