Skip to content
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

chore: handle upgrades #1174

Merged
merged 2 commits into from
Mar 21, 2024
Merged

Conversation

jmeridth
Copy link
Contributor

@jmeridth jmeridth commented Mar 21, 2024

Was originally in #1172

Also:

  • remove safe option from pyproject black config
    • it is the default, so no need to state it in config
    • fast is the non-default option and works when added to config without error (docs)
    • Error being seen was:
      Invalid config keys detected: 'safe' (in code/github3.py/pyproject.toml)
  • add venv to excluding directories (most devs set their virtualenv to venv or .venv)

Before opening a new issue, please search for a previously filed
issue to ensure you're not creating a duplicate.

Note Bug reports without the following will receive requests for these
details to be provided.

Version Information

Please provide:

  • The version of Python you're using (3.12.2)

  • The version of pip you used to install github3.py (24.0)

  • The version of github3.py (4.0.1), requests (2.31.0), uritemplate (4.1.1), and python-dateutil (2.9.0-post0) installed

Minimum Reproducible Example

Please provide an example of the code that generates the error you're seeing.

Exception information

What is exceptional about what you're seeing versus what you expected to see.

jmeridth and others added 2 commits March 21, 2024 07:07
Was originally in sigmavirus24#1172

Also:

- [x] remove safe option from pyproject black config
  - it is the default, so no need to state, stop config error notification
  - `fast` is the non-default option and works when added to config without
    error ([docs](https://black.readthedocs.io/en/stable/usage_and_configuration/the_basics.html#fast-safe))
- [x] add venv to excluding directories (most devs set their virtualenv to venv or .venv)

Signed-off-by: jmeridth <jmeridth@gmail.com>
@jmeridth jmeridth changed the title chore: hande upgrades chore: handle upgrades Mar 21, 2024
@sigmavirus24 sigmavirus24 merged commit 3ae8761 into sigmavirus24:main Mar 21, 2024
11 checks passed
@jmeridth jmeridth deleted the jm-handle-upgrades branch March 21, 2024 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants