Skip to content

[TASK] Drop redundant assignment in runTests.sh#1962

Merged
kanow merged 1 commit intomainfrom
cleanup/redundant-assignment
Feb 6, 2026
Merged

[TASK] Drop redundant assignment in runTests.sh#1962
kanow merged 1 commit intomainfrom
cleanup/redundant-assignment

Conversation

@oliverklee
Copy link
Contributor

The assign variable is overwritten in all branches of the following condition.

Fixes #1959

@coveralls
Copy link

coveralls commented Feb 2, 2026

Pull Request Test Coverage Report for Build 21720036412

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 21719794939: 0.0%
Covered Lines: 111
Relevant Lines: 111

💛 - Coveralls

The assign variable is overwritten in all branches of the
following condition.

Fixes #1959
@oliverklee oliverklee force-pushed the cleanup/redundant-assignment branch from eeb0531 to ddef27d Compare February 5, 2026 16:39
Copy link
Contributor

@kanow kanow left a comment

Choose a reason for hiding this comment

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

Yes, that make sanse.

@kanow kanow merged commit 38dcb30 into main Feb 6, 2026
32 checks passed
@kanow kanow deleted the cleanup/redundant-assignment branch February 6, 2026 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Drop redundant assignment in runTests.sh

3 participants