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.

Sign up for GitHub

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

Jump to bottom

Add the ClearlyDefined certifier to the demo compose file #2129

Merged
merged 1 commit into from
Sep 11, 2024
Merged

Add the ClearlyDefined certifier to the demo compose file #2129

merged 1 commit into from
Sep 11, 2024

Conversation

funnelfiasco
Copy link
Contributor

Fixes #2128

Description of the PR

PR Checklist

  • All commits have a Developer Certificate of Origin (DCO) -- they are generated using -s flag to git commit.
  • All new changes are covered by tests
  • If GraphQL schema is changed, make generate has been run
  • If GraphQL schema is changed, GraphQL client updates/additions have been made
  • If OpenAPI spec is changed, make generate has been run
  • If ent schema is changed, make generate has been run
  • If collectsub protobuf has been changed, make proto has been run
  • All CI checks are passing (tests and formatting)
  • All dependent PRs have already been merged

@funnelfiasco
Add the ClearlyDefined certifier to the demo compose file
9a7e519 
Fixes guacsec#2128

Signed-off-by: Ben Cotton <ben@kusari.dev>
@kodiakhq kodiakhq bot merged commit c22cf02 into guacsec:main Sep 11, 2024
8 checks passed
funnelfiasco added a commit to funnelfiasco/guac that referenced this pull request Sep 13, 2024
@funnelfiasco
Also add the ClearlyDefined certifier to the postgres compose file
9757b67 
Similar to what was done in guacsec#2129

Fixes guacsec#2128
(Even though that's already closed as fixed. Oops!)

Signed-off-by: Ben Cotton <ben@kusari.dev>
funnelfiasco added a commit to funnelfiasco/guac that referenced this pull request Sep 13, 2024
@funnelfiasco
Also add the ClearlyDefined certifier to the postgres compose file
b3c6611 
Similar to what was done in guacsec#2129

Fixes guacsec#2128
(Even though that's already closed as fixed. Oops!)

Signed-off-by: Ben Cotton <ben@kusari.dev>
@funnelfiasco funnelfiasco mentioned this pull request Sep 13, 2024
Merged
9 tasks
kodiakhq bot pushed a commit that referenced this pull request Sep 18, 2024
@funnelfiasco
Also add the ClearlyDefined certifier to the postgres compose file (#…
d4abef2 
…2130)

Similar to what was done in #2129

Fixes #2128
(Even though that's already closed as fixed. Oops!)

Signed-off-by: Ben Cotton <ben@kusari.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pxp928 pxp928 pxp928 approved these changes

@jeffmendoza jeffmendoza jeffmendoza approved these changes

Assignees
No one assigned
Labels
size/S
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[feature] Add the ClearlyDefined certifier to the demo compose file
3 participants
@funnelfiasco @jeffmendoza @pxp928