-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
chore: updated dependencies for examples #3057
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3057 +/- ##
==========================================
- Coverage 82.65% 82.64% -0.01%
==========================================
Files 62 62
Lines 12856 12856
==========================================
- Hits 10626 10625 -1
- Misses 1691 1692 +1
Partials 539 539
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
94e0c16
to
ab8dc0d
Compare
ab8dc0d
to
dadb3c7
Compare
b8643be
to
7dd7179
Compare
* updated dependencies for oauth2 examples: oidc and jwt new versions came with breaking changes * ci: added build check for examples > NOTE: initially, I wanted to reduce the set of main dependencies by making the "examples" folder > it own module. Eventually, this did not change much (apart oidc and jwt) > and was adding more maintainance burden. Given up on that. Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
7dd7179
to
c1a3b10
Compare
@@ -58,6 +58,12 @@ jobs: | |||
run: | | |||
go install ./cmd/swagger | |||
|
|||
- name: Verify that examples build |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't you think it could deserve it's own job in the workflow with a dedicated step for each these commands ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know. This would require a "workflow_run" trigger to operate after the previous steps, but yes, essentually why not? If you feel strongly about that, please propose a follow-up PR.
came with breaking changes
Signed-off-by: Frederic BIDON fredbi@yahoo.com