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

[improve][ci] Run CodeQL within Pulsar CI workflow as mandatory check #22132

Merged
merged 5 commits into from
Feb 28, 2024
Merged

[improve][ci] Run CodeQL within Pulsar CI workflow as mandatory check #22132

merged 5 commits into from
Feb 28, 2024

Conversation

lhotari
Copy link
Member

@lhotari lhotari commented Feb 27, 2024 •
edited
Loading

Motivation & Modifications

Improvements to CI:

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Feb 27, 2024
@lhotari lhotari force-pushed the lh-enable-ci-for-branches branch from 4ceee8f to 4aeee34 Compare February 27, 2024 09:59
@lhotari lhotari changed the title [improve][ci] Enable Pulsar CI for branches, cancel duplicates for CodeQL [improve][ci] Run CodeQL within Pulsar CI workflow for PRs so that it is mandatory Feb 27, 2024
@lhotari lhotari requested a review from asafm February 27, 2024 13:34
@lhotari lhotari changed the title [improve][ci] Run CodeQL within Pulsar CI workflow for PRs so that it is mandatory [improve][ci] Run CodeQL within Pulsar CI workflow as mandatory check Feb 27, 2024
@lhotari lhotari requested a review from merlimat February 27, 2024 16:45
@lhotari lhotari requested a review from heesung-sn February 27, 2024 17:10
@lhotari lhotari merged commit 86dc039 into apache:master Feb 28, 2024
55 of 57 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@heesung-sn heesung-sn heesung-sn approved these changes

@Technoboy- Technoboy- Awaiting requested review from Technoboy-

@asafm asafm Awaiting requested review from asafm

@merlimat merlimat Awaiting requested review from merlimat

Assignees
No one assigned
Labels
area/ci doc-not-needed Your PR changes do not impact docs ready-to-test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lhotari @heesung-sn