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
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
Copy link
Copy Markdown

linear-code Bot commented May 18, 2026

LIN-70776

Copy link
Copy Markdown

@linear-code linear-code Bot left a comment

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 Add broader for matching issues identifiers 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants