New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve scons docs regarding Default() #3261

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@mwichmann
Copy link
Contributor

mwichmann commented Jan 7, 2019

An earlier change updated some examples, this makes some tweaks to the wording as well for Default, DEFAULT_TARGETS, etc. to clarify.

This is a doc-only change (no code or test impact). It represents an opinion on slightly clearer wording, but is not really fixing anything "broken".

Signed-off-by: Mats Wichmann mats@linux.com

Contributor Checklist:

  • I have created a new test or updated the unit tests to cover the new/changed functionality.
  • I have updated master/src/CHANGES.txt directory (and read the README.txt in that directory)
  • I have updated the appropriate documentation
@coveralls

This comment has been minimized.

Copy link

coveralls commented Jan 7, 2019

Coverage Status

Coverage increased (+0.8%) to 80.824% when pulling 8358959 on mwichmann:doc-default into d36a52a on SCons:master.

3 similar comments
@coveralls

This comment has been minimized.

Copy link

coveralls commented Jan 7, 2019

Coverage Status

Coverage increased (+0.8%) to 80.824% when pulling 8358959 on mwichmann:doc-default into d36a52a on SCons:master.

@coveralls

This comment has been minimized.

Copy link

coveralls commented Jan 7, 2019

Coverage Status

Coverage increased (+0.8%) to 80.824% when pulling 8358959 on mwichmann:doc-default into d36a52a on SCons:master.

@coveralls

This comment has been minimized.

Copy link

coveralls commented Jan 7, 2019

Coverage Status

Coverage increased (+0.8%) to 80.824% when pulling 8358959 on mwichmann:doc-default into d36a52a on SCons:master.

@coveralls

This comment has been minimized.

Copy link

coveralls commented Jan 7, 2019

Coverage Status

Coverage increased (+0.5%) to 80.542% when pulling ecd8ec2 on mwichmann:doc-default into 0f782d0 on SCons:master.

@bdbaddog bdbaddog changed the title Work on Default docu wording Improve scons docs regarding Default() Jan 8, 2019

@mwichmann mwichmann force-pushed the mwichmann:doc-default branch from 8358959 to 1cf0857 Jan 11, 2019

Work on Default docu wording a bit more
An earlier change updated some examples, this makes some tweaks to
the wording as well for Default, DEFAULT_TARGETS, etc. to clarify.

Signed-off-by: Mats Wichmann <mats@linux.com>

@mwichmann mwichmann force-pushed the mwichmann:doc-default branch from 1cf0857 to ecd8ec2 Jan 14, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment