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

Update .pre-commit-config.yaml #9502

Merged
merged 1 commit into from May 9, 2023
Merged

Update .pre-commit-config.yaml #9502

merged 1 commit into from May 9, 2023

Conversation

mosfet80
Copy link
Contributor

@mosfet80 mosfet80 commented May 7, 2023

update pre-commit external libraries

Thank you for creating a pull request to contribute to FreeCAD! Place an "X" in between the brackets below to "check off" to confirm that you have satisfied the requirement, or ask for help in the FreeCAD forum if there is something you don't understand.

  • Your Pull Request meets the requirements outlined in section 5 of CONTRIBUTING.md for a Valid PR

Please remember to update the Wiki with the features added or changed once this PR is merged.
Note: If you don't have wiki access, then please mention your contribution on the 1.0 Changelog Forum Thread.


@chennes
Copy link
Member

chennes commented May 7, 2023

Don't change the major versions without discussing it in an Issue first. These versions were chosen deliberately.

@adrianinsaval
Copy link
Member

any particular reason why these should be updated?

@mosfet80
Copy link
Contributor Author

mosfet80 commented May 8, 2023

Fix incorrectly applied .gitignore rules by considering the .gitignore location
and the relative path to the target file (psf/black#3338)
Fix incorrectly ignoring .gitignore presence when more than one source directory is
specified (psf/black#3336)

update pre-commit external libraries
Update .pre-commit-config.yaml
@chennes chennes merged commit 87756b4 into FreeCAD:master May 9, 2023
6 checks passed
@mosfet80 mosfet80 deleted the patch-3 branch August 21, 2023 07:46
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

3 participants