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

Modified deprecated projectID field of Gitlab Eventsource to be optional #1500

Conversation

daniel-codefresh
Copy link
Member

@daniel-codefresh daniel-codefresh commented Jan 5, 2022

Signed-off-by: Daniel Soifer [email protected]

Fixes the following JSON schema validation error when using projects field instead of the deprecated projectID field in a Gitlab Eventsource:

Screen Shot 2022-01-05 at 14 41 55

@whynowy whynowy merged commit faa69c7 into argoproj:master Jan 5, 2022
BulkBeing pushed a commit to BulkBeing/argo-events that referenced this pull request Mar 7, 2022
…nal (argoproj#1500)

* modified deprecated projectID field of gitlab eventsource to be optional

Signed-off-by: Daniel Soifer <[email protected]>

* modified deprecated projectID field of gitlab eventsource to be optional

Signed-off-by: Daniel Soifer <[email protected]>

* fix: Modified deprecated projectID field of gitlab eventsource to be optional

Signed-off-by: Daniel Soifer <[email protected]>

* codegen

Signed-off-by: Daniel Soifer <[email protected]>
juliev0 pushed a commit to juliev0/argo-events that referenced this pull request Mar 29, 2022
…nal (argoproj#1500)

* modified deprecated projectID field of gitlab eventsource to be optional

Signed-off-by: Daniel Soifer <[email protected]>

* modified deprecated projectID field of gitlab eventsource to be optional

Signed-off-by: Daniel Soifer <[email protected]>

* fix: Modified deprecated projectID field of gitlab eventsource to be optional

Signed-off-by: Daniel Soifer <[email protected]>

* codegen

Signed-off-by: Daniel Soifer <[email protected]>
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