-
Notifications
You must be signed in to change notification settings - Fork 35
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
Add emissary ingress CRD #123
base: master
Are you sure you want to change the base?
Conversation
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.
Please provide tests and a readme file.
@@ -0,0 +1,48 @@ | |||
{ | |||
"name": "@kubernetes-models/emissary-ingress", | |||
"version": "3.6.0", |
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.
"version": "3.6.0", | |
"version": "0.0.0", |
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.
Oh, should this version not match the version that is provided in the CRDs? That was my understanding of it
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.
The version doesn't reflect the CRD version.
Co-authored-by: Tommy Chen <[email protected]>
@alexbowers are you still interested in this? If not, I could also take a look as part of a hacktoberfest contribution. |
Yes, for now we have a backyard solution in place
…________________________________
From: Christoph Jerolimov ***@***.***>
Sent: Tuesday, October 24, 2023 7:28:49 AM
To: tommy351/kubernetes-models-ts ***@***.***>
Cc: Alex Bowers ***@***.***>; Mention ***@***.***>
Subject: Re: [tommy351/kubernetes-models-ts] Add emissary ingress CRD (PR #123)
@alexbowers<https://github.com/alexbowers> are you still interested in this? If not, I could also take a look as part of a hacktoberfest contribution.
—
Reply to this email directly, view it on GitHub<#123 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AAGNZXV6JTALB6GZDFQSMJLYA5N2DAVCNFSM6AAAAAAXO55GEGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZWGYYDKNZRGQ>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Sorry, I need more clarity. You are interested, but will you update the PR based on the feedback (version number, readme, tests), or should I try to open a new one? |
To be honest, I don't actually use Kubernetes Models TS anymore, i've instead moved to using CDK8s; so feel free to take this on, or close the PR and make your own |
No description provided.