William Deegan

Results 323 comments of William Deegan

I don't think worth huge change. But if there's a way to give a clearer error message, that's probably worth persuing?

Nope. I think this is an enhancement request asking for configure checks: - `conf.CheckFile(filename)` which would check if the given file exists - `conf.CheckProgram(programName)` which would check if the given...

o.k. yes doc for manpage for Detect().. but the .exists shouldn't be in manpage but rather in API docs via docstring.

Hmm. there's https://github.com/SCons/scons/wiki/ToolsForFools and/or this: https://scons.org/doc/production/HTML/scons-user.html#idm3860

I guess the bigger question is the issue resolved and thus removing the need to delete this bit of doc?

What happens if the user option is not boolean, say a string and we allow SetOption()? (As in what kind of bad?)

The version # of the next release is in flux. But please go ahead and work on this. It'll make it into the next release after you complete with requisite...

> Any traction on this ? Not yet. It's on the list, just requires a big chunk of time to review and likely modify before being able to pull into...

> We've faced the same issue today => https://github.com/platformio/platformio-core/actions/runs/2345754750 > > We treat Python warnings as errors in https://github.com/platformio/platformio-core/blob/develop/pytest.ini > > It would be great to have this fixed in...

@ivankravets - Please PR. We'll work from there?