Skip to content

ci: Add CI step to check for draftlog#7850

Merged
camdecoster merged 3 commits into
masterfrom
cam/7849/add-ci-check-draftlog
Jun 18, 2026
Merged

ci: Add CI step to check for draftlog#7850
camdecoster merged 3 commits into
masterfrom
cam/7849/add-ci-check-draftlog

Conversation

@camdecoster

@camdecoster camdecoster commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Description

Add CI step to check for a draftlog on a PR.

Closes #7849.

Changes

  • Add workflow
  • Update docs

Testing

Check CI after this has been merged on future PRs.

Notes

  • This check can be skipped by applying the no-draftlog label to the PR in GitHub

@camdecoster camdecoster added the no-draftlog The associated PR doesn't require a draftlog and the CI check will be skipped. label Jun 18, 2026
@camdecoster

Copy link
Copy Markdown
Contributor Author

The "Check Draftlog" step failed without the label and passed with the label.

Comment on lines +7 to +9
concurrency:
group: check-draftlog-${{ github.event.pull_request.number }}
cancel-in-progress: true

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Is this needed?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It's there to cover when multiple commits get pushed in quick succession.

Comment thread .github/workflows/check-draftlog.yml Outdated
Comment thread .github/workflows/check-draftlog.yml Outdated
emilykl
emilykl previously approved these changes Jun 18, 2026
Co-authored-by: Emily KL <4672118+emilykl@users.noreply.github.com>
@camdecoster camdecoster merged commit 06f8544 into master Jun 18, 2026
30 of 32 checks passed
@camdecoster camdecoster deleted the cam/7849/add-ci-check-draftlog branch June 18, 2026 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-draftlog The associated PR doesn't require a draftlog and the CI check will be skipped.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE]: Add CI step to check for draftlog in PR

2 participants