Skip to content

Fixups to commit message checks to only happen during PR pushes - #509

Merged
Brian Kroth (bpkroth) merged 8 commits into
microsoft:mainfrom
bpkroth:ci-fixups-2
Sep 18, 2023
Merged

Fixups to commit message checks to only happen during PR pushes#509
Brian Kroth (bpkroth) merged 8 commits into
microsoft:mainfrom
bpkroth:ci-fixups-2

Conversation

@bpkroth

Copy link
Copy Markdown
Contributor

Handles bugs from #508.

Also try and fix a quoting issues.

@bpkroth
Brian Kroth (bpkroth) requested a review from a team as a code owner September 18, 2023 19:28
@bpkroth
Brian Kroth (bpkroth) marked this pull request as draft September 18, 2023 19:29
@bpkroth
Brian Kroth (bpkroth) marked this pull request as ready for review September 18, 2023 19:42
@bpkroth
Brian Kroth (bpkroth) enabled auto-merge (squash) September 18, 2023 19:42
@bpkroth Brian Kroth (bpkroth) added the ready for review Ready for review label Sep 18, 2023

@motus Sergiy Matusevych (motus) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perfect, thanks!

@bpkroth
Brian Kroth (bpkroth) merged commit 0b0842f into microsoft:main Sep 18, 2023
Brian Kroth (bpkroth) added a commit to bpkroth/MLOS that referenced this pull request Sep 19, 2023
Brian Kroth (bpkroth) added a commit that referenced this pull request Sep 20, 2023
One more minor tweak on #509 and #508. This allows us to force a rebuild
of the devcontainer with NO_CACHE=true manually without needing to push
an empty commit.
@bpkroth
Brian Kroth (bpkroth) deleted the ci-fixups-2 branch October 10, 2023 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready for review Ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants