chore/py311 #3
Closed
chore/py311 #3
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Remove direct dependency on markupsafe
pre-commit: add bandit hook
Closes: #1523
Force click<8.0.0
add safety ci step
ensure filesystem isolation during tests execution
Make release-drafter diff only between master releases
Adopt setuptools-scm packaging
* Make read_user_dict compatible with click 8.x
The `read_user_dict` function uses a "default" sentinel instead of the actual
default value. Being a JSON dict, the latter would often be hard to type.
Under click 8.x, the default value for `click.prompt` is passed to the
`read_proc` callback. We use this callback to load JSON from the user input, and
this would choke on an input like "default" (without quotes). Therefore, change
the callback to return the default value when it receives the "default"
sentinel.
Under click 7.x (which is our minimum version), the default value for
`click.prompt` is returned as-is. Therefore, continue to handle the case where
`click.prompt` returns "default" instead of the actual default value, but only
if we're actually running under click 7.x.
(Checking for click 7.x is only done for clarity. Under click 8.x,
`click.prompt` would never return "default", even if a user entered it as a
valid JSON string. This is because our callback requires a dict, not a string.)
* test: Expect read_user_dict to call click.prompt with partial object
Previously, tests for `read_user_dict` expected `process_json` to be passed to
click.prompt directly. Instead, we now pass an instance of `functools.partial`,
so adapt the mock to reflect that.
* test: Avoid mocking `click.prompt` when testing defaults
Do not mock `click.prompt` when testing that `read_user_dict` returns the proper
default value (rather than the sentinel "default"). Mocking `click.prompt`
prevents our callback from running, and under click >= 8.0 we process the
sentinel in the callback. Instead, use `click.testing.CliRunner` to fake
standard input.
* test: Adapt regression test for default handling
Expect `json.loads` not to be called with the sentinel ("default"). Previously,
the test expected `process_json` not to be called, but under click >= 8.0 that
is now where we handle the sentinel.
* Update dependencies for click 8.x
* Add test cases for untested branches * Add branches coverage Branch coverage helps to find out untested branches. Closes: #1541
Relates to #1543
- Name corrections for @audreyfeldroy and @pydanny - Run Prettier on Markdown files
- Resolves GitHub check failures due to GitHub no longer supporting those environments on Mac OSX. - Python 3.6 is in maintenance status *security* with EOL in 2 months. - PyPy was last updated to Python 3.8, which is outdated.
Co-authored-by: Daniel Roy Greenfeld <pydanny@gmail.com>
- Update Black version - Use Black python target of Python 3.9 Co-authored-by: Daniel Roy Greenfeld <pydanny@gmail.com>
Code quality: Core files: Added exception reason reraise when exception class changed (PEP 3134)
…hlib Code quality: Tests upgrade: Use pathlib for files read/write
…ake_sure_path_exists
Co-authored-by: Jens W. Klein <jk@kleinundpartner.at>
Code quality: find.py refactored and type annotated
…ts_and_related_code Code quality: utils.make_sure_path_exists refactored and type annotated
Fix typos and add minor doc updates
There is a small typo in docs/conf.py. Should read `that` rather than `shat`.
Authored-by: Segun Babalola <s.babalola@ucl.ac.uk>
…xt (#1771) * Changes to cookiecutter/main.py: * Add resolved template repository path as _repo_dir to the context * Changes to tests/test_cli.py: * corrected test test_echo_undefined_variable_error to check for new context item _repo_dir Co-authored-by: Thomas Meckel <tmeckel@users.noreply.github.com>
- enable py311 pipelines - expose py311 support in metadata
ssbarnea
force-pushed
the
chore/py311
branch
from
678b448
to
88135c1
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
--no-inputdocumentation cookiecutter/cookiecutter#1543)no_inputflag