Skip to content

Conversation

mwichmann
Copy link
Collaborator

Two more chunk moves, to get closer to other diffs being useful. These are simpler than the previous change, and don't confuse the diff tools, but the second at least would get buried if the change were bigger.

  • the warning about chdir behavior is moved to after the example of a builder using chdir.
  • two separate paragraphs, one at line 3274 of the original, the other at line 3333, are combined into one and moved up to the place where the concept of builders returning a NodeList is introduced.

No code changes. Changelog content coverted by part 1 (PR #4762).

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

Two more chunk moves, to get closer to other diffs being useful.
These are simpler than the previous change, and don't confuse the
diff tools, but the second at least would get buried if the change
were bigger.

* the warning about 'chdir' behavior is moved to after the example
  of a builder using chdir.
* two separate paragraphs, one at line 3274 of the original, the other
  at line 3333, are combined into one and moved up to the place
  where the concept of builders returning a NodeList is introduced.

Signed-off-by: Mats Wichmann <[email protected]>
@mwichmann mwichmann added this to 4.10 Sep 11, 2025
@mwichmann mwichmann added documentation testsuite Things that only affect the SCons testing. Do not use just because a PR has tests. labels Sep 11, 2025
@mwichmann mwichmann moved this to In review in 4.10 Sep 11, 2025
@mwichmann mwichmann changed the title Builder method man updates, part 2 [skip appveyor] Builder method man updates, part 2 Sep 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation testsuite Things that only affect the SCons testing. Do not use just because a PR has tests.
Projects
Status: In review
Development

Successfully merging this pull request may close these issues.

1 participant