Flight Log / MikePopoloski/slang #1372
[slang-tidy] Fix RegisterHasNoReset false warning for loop iterator variables — a GitHub Copilot (GitHub) bot flight
The GitHub Copilot bot (GitHub) opened this pull request in MikePopoloski/slang on May 24, 2025; merged clean in 9.9 h, merged by @MikePopoloski: +43 −6, 3 files, 3 reviews.
Flown by
- Lines
- +43 −6
- Files
- 3
- Commits
- 3
- Reviews
- 3
- To merge
- 9.9 h
- Task
- fix
Line and file counts are summed per commit from the AIDev dataset; GitHub's net diff can be smaller.
GitHub Copilot submitted a fix to slang's RegisterHasNoReset tidy check, which wrongly warned on temporary loop iterator variables used in always_ff blocks. The change touched three files across three commits, adding 43 lines and removing six. The PR merged after 9.9 hours with zero change requests and four review comments.
From the pull request description
This PR fixes an issue with the `RegisterHasNoReset` tidy check incorrectly triggering warnings for loop iterator variables used in `always_ff` blocks. The warning is intended for actual registers that don't have reset values, but was incorrectly triggering for temporary loop variables. ## Problem …
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#1372, record updated Nov 4, 2026, https://agentriders.com/logs/MikePopoloski/slang/1372/
Data from public GitHub activity; this page is not affiliated with GitHub or the repository.