Flight Log / MikePopoloski/slang #1371
Fix OnlyAssignedOnReset false positive with struct arrays and for loops — a GitHub Copilot (GitHub) bot flight
The GitHub Copilot bot (GitHub) opened this pull request in MikePopoloski/slang on May 23, 2025; merged clean in 7.2 h, merged by @MikePopoloski: +88 −48, 3 files, 3 reviews.
Flown by
- Lines
- +88 −48
- Files
- 3
- Commits
- 3
- Reviews
- 3
- To merge
- 7.2 h
- Task
- fix
Line and file counts are summed per commit from the AIDev dataset; GitHub's net diff can be smaller.
GitHub Copilot fixed a false positive in the OnlyAssignedOnReset lint check in slang, which flagged struct array elements assigned in a for loop within an always_ff non-reset branch. Across three files, 88 lines were added and 48 removed. It merged clean after one review round, with no change requests, in 7.21 hours.
From the pull request description
## Issue The `OnlyAssignedOnReset` lint check was incorrectly flagging struct array elements assigned within a for loop in the non-reset branch of an always_ff block. This resulted in false positives when struct arrays were properly assigned in both reset and non-reset conditions. For example, giv…
Excerpt (first 300 characters) — the full description is on GitHub.
View the pull request on GitHub →
This is your flight? Claim it — sign in with GitHub. Ownership is proven by the sign-in: the account that opened the PR is the only one that can claim it.
Flight timeline
- —committedcopilot-swe-agent[bot]
- —committedcopilot-swe-agent[bot]
- —reviewed
- —committedcopilot-swe-agent[bot]
- May 24, 2025mergedMikePopoloski
- May 24, 2025closedMikePopoloski
Read next
Cite this flight: AgentRiders Flight Log, MikePopoloski/slang#1371, record updated Nov 4, 2026, https://agentriders.com/logs/MikePopoloski/slang/1371/
Data from public GitHub activity; this page is not affiliated with GitHub or the repository.