-
Notifications
You must be signed in to change notification settings - Fork 29
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
Refactor this issuer to use issuer-lib #109
Conversation
a2cbd69
to
d72926c
Compare
d72926c
to
d875db5
Compare
40c52b3
to
baa16f4
Compare
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.
It doesn't look like the e2e test is running, which is really they key in evaluating whether this is safe or not.
IIRC it doesn't run on forks. Could you maybe re-create this PR from a branch on the jetstack/google-cas-issuer repo directly, which I think should trigger the E2E?
@SgtCoDFish I just had to add the |
Signed-off-by: Tim Ramlot <42113979+inteon@users.noreply.github.com>
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.
/lgtm
/approve
Great stuff! I guess next step is to release an alpha version? 🚀
This is the first open-source issuer that will start using this library.
This change greatly simplifies this issuer.