Skip to content

Bool tweaks - #4946

Merged
bdbaddog merged 2 commits into
SCons:masterfrom
mwichmann:maint/bools
Nov 11, 2026
Merged

bdbaddog merged 2 commits into
SCons:masterfrom
mwichmann:maint/bools

Conversation

@mwichmann

Copy link
Copy Markdown
Collaborator

A few more variables which were used like a bool (i.e. only truthiness tests) switched to use True/False instead of 1/0.

Clarified comments on a couple of these variables defined in a module, but reached in and modified at startup from Main.py.

There are no functional impacts, so no doc or test changes.

Contributor Checklist:

  • I have created a new test or updated the unit tests to cover the new/changed functionality.
  • I have updated CHANGES.txt and RELEASE.txt (and read the README.rst).
  • I have updated the appropriate documentation

A few more variables which were used like a bool (i.e. only truthiness
tests) switched to use True/False instead of 1/0.

Clarified comments on a couple of these variables defined in a module,
but reached in and modified at startup from Main.py.

Signed-off-by: Mats Wichmann <mats@linux.com>
@mwichmann mwichmann added the maintenance Tasks to maintain internal SCons code/tools label Nov 9, 2026
@mwichmann mwichmann moved this to In review in Next Release Nov 10, 2026
Comment thread test/implicit-cache/SetOption.py Resolved
@bdbaddog
bdbaddog merged commit 418367d into SCons:master Nov 11, 2026
12 of 13 checks passed
@github-project-automation github-project-automation Bot moved this from In review to Done in Next Release Nov 11, 2026
@mwichmann mwichmann added this to the NextRelease milestone Nov 11, 2026
@mwichmann
mwichmann deleted the maint/bools branch November 11, 2026 23:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Tasks to maintain internal SCons code/tools

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants