VictorColomb opened a new pull request, #9336:
URL: https://github.com/apache/devlake/pull/9336

   ### ⚠️ Pre Checklist
   
   - [x] I have read through the [Contributing 
Documentation](https://devlake.apache.org/community/).
   - [x] I have added relevant tests.
   
   ### Summary
   `LinkPrToIssue` compared extracted keys to `issues.issue_key` with an exact, 
case-sensitive match, so lowercase branch names like `user/proj-123-title` 
never linked to `PROJ-123` (on case-sensitive DBs such as PostgreSQL).
   
   - `normalizeIssueKey` now uppercases the extracted key.
   - The query compares `UPPER(issues.issue_key)`, so keys stored in lowercase 
also match.
   - Added e2e test `TestLinkPrToIssueCaseInsensitive`. It fails without the 
fix on PostgreSQL; on MySQL the default collation is already case-insensitive, 
so it passes either way.
   
   Not addressed: the "first matching field wins" limitation mentioned in the 
issue.
   
   ### Does this close any open issues?
   Closes #9283


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to