Skip to content

Add broader for matching issues identifiers#81

Merged
axelniklasson merged 3 commits into
mainfrom
axel/lin-70776-lin-123-my-change-does-not-pick-up-lin-123-as-an-issue
May 18, 2026
Merged

Add broader for matching issues identifiers#81
axelniklasson merged 3 commits into
mainfrom
axel/lin-70776-lin-123-my-change-does-not-pick-up-lin-123-as-an-issue

Conversation

@axelniklasson

@axelniklasson axelniklasson commented May 18, 2026

Copy link
Copy Markdown
Collaborator

Updates the commit scanning to also pick up issue identifiers in commits when the identifier is enclosed in brackets, such as [ENG-123] My change. We support this when linking diffs and issues, so makes sense to also support this here.

Relates to #71
@linear-code

linear-code Bot commented May 18, 2026

Copy link
Copy Markdown

LIN-70776
linear-code[bot]
linear-code Bot reviewed May 18, 2026
View reviewed changes

@linear-code linear-code Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good. The bracket-extraction logic is correctly scoped to the subject line only, the revert-depth guard (afterTitle when odd-depth) prevents false positives on reverted commits, and the leading-zero guard is sound. No blocking issues found.
@axelniklasson axelniklasson requested a review from RomainCscn May 18, 2026 09:22
@axelniklasson axelniklasson changed the title Add support for matching issues identifiers in brackets May 18, 2026
@axelniklasson axelniklasson enabled auto-merge (squash) May 18, 2026 10:02
@axelniklasson axelniklasson merged commit 6367b71 into main May 18, 2026
10 checks passed
@axelniklasson axelniklasson deleted the axel/lin-70776-lin-123-my-change-does-not-pick-up-lin-123-as-an-issue branch May 18, 2026 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@axelniklasson @RomainCscn