summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #1590 from Hawk777/init-exceptionsHEADmainSebastian Thiel2023-05-151-4/+4
|\ | | | | Name top-level exceptions as private variables
| * Name top-level exceptions as private variablesChristopher Head2023-05-151-4/+4
|/ | | | | | `exc` is private to the module. Naming it `_exc` eliminates a collision with the `exc` submodule (one which would not be observable at runtime due to the import failing, but which confuses linters).
* Merge pull request #1576 from itsluketwist/fix-trailersSebastian Thiel2023-04-232-58/+121
|\ | | | | Fix up the commit trailers functionality
| * Revert the removal of Commit.trailers property.Twist2023-04-231-0/+14
| |
| * Specify encoding in Commit.trailer_list.Twist2023-04-221-1/+1
| |
| * Update Commit.trailer_list to return tuples.Twist2023-04-222-15/+20
| |
| * Deprecate Commit.trailers.Twist2023-04-222-24/+0
| |
| * Add trailers_list and trailers_list methods to fix the commit trailers ↵Twist2023-04-212-58/+126
|/ | | | functionality. Update trailers tests.
* Merge pull request #1572 from ↵Sebastian Thiel2023-04-101-1/+1
|\ | | | | | | | | gitpython-developers/dependabot/github_actions/cygwin/cygwin-install-action-4 Bump cygwin/cygwin-install-action from 3 to 4
| * Bump cygwin/cygwin-install-action from 3 to 4dependabot[bot]2023-04-101-1/+1
|/ | | | | | | | | | | | | | Bumps [cygwin/cygwin-install-action](https://github.com/cygwin/cygwin-install-action) from 3 to 4. - [Release notes](https://github.com/cygwin/cygwin-install-action/releases) - [Commits](https://github.com/cygwin/cygwin-install-action/compare/v3...v4) --- updated-dependencies: - dependency-name: cygwin/cygwin-install-action dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
* prepare next release3.1.31Sebastian Thiel2023-02-162-1/+7
|
* Merge pull request #1555 from Codym48/fix/get_valuesSebastian Thiel2023-02-162-0/+12
|\ | | | | Fix get_values() so it correctly loads section names
| * Add test asserting that get_values works by itselfCody Martin2023-02-151-0/+11
| | | | | | | | | | | | | | | | | | | | | | As described in #1534, this test will fail in main with a KeyError about a missing section name even though the named sections do exist within the config file. This test will pass in the branch associated with #1535. This test should keep the improved behavior healthy as the code evolves by preventing future developers from removing the eager loading of sections within get_values().
| * get_values eagerly loads sections before returnKent Friesen2023-01-101-0/+1
| |
* | Merge pull request #1554 from eric-wieser/patch-4Sebastian Thiel2023-02-121-3/+2
|\ \ | | | | | | Fix RecursionError when iterating streams
| * | Update cmd.pyEric Wieser2023-02-121-2/+1
| | |
| * | Fix RecursionError when iterating streamsEric Wieser2023-02-121-3/+3
|/ /
* | Merge pull request #1550 from Sineaggi/remove-optional-from-two-variablesSebastian Thiel2023-02-022-29/+36
|\ \ | | | | | | Remove optional from two member variables
| * | Update docsClayton Walker2023-02-021-1/+1
| | |
| * | Add additional assertions to test_base.pyClayton Walker2023-02-021-2/+6
| | |
| * | Add check to test bare repoClayton Walker2023-02-011-0/+1
| | |
| * | Remove optional from two member variablesClayton Walker2023-01-271-26/+28
| | |
* | | Merge pull request #1551 from mellowed100/mainSebastian Thiel2023-02-022-1/+74
|\ \ \ | |/ / |/| | Enable user to override default diff -M arg
| * | Updated diff test to use different similarity thresholdsCesar Velazquez2023-02-011-3/+26
| | |
| * | Added diff test to disable rename detectionCesar Velazquez2023-01-311-0/+47
| | |
| * | fixed lint errorCesar Velazquez2023-01-301-1/+1
| | |
| * | Enable user to override default diff -M argCesar Velazquez2023-01-301-1/+4
|/ /
* | Merge pull request #1547 from jcowgill/non-hour-timezonesSebastian Thiel2023-01-252-16/+40
|\ \ | | | | | | Fix timezone parsing functions for non-hour timezones
| * | Fix timezone parsing functions for non-hour timezonesJames Cowgill2023-01-242-16/+40
|/ / | | | | | | | | | | | | | | The `utctz_to_altz` and `altz_to_utctz_str` functions fail to handle timezones with UTC offsets that are not a multiple of one hour. Rewrite them and add some unit tests. Fixes #630
* | Merge pull request #1545 from Lightborne/fix_ignoredSebastian Thiel2023-01-222-2/+36
|\ \ | | | | | | Fix ignored
| * | Raise exception if return code from check-ignore is not 1Axel Aguado2023-01-211-2/+9
| | |
| * | Add test to verify GitCommandError is raised when check-ignore is run ↵Axel Aguado2023-01-211-1/+12
| | | | | | | | | | | | against a file behind a symlink
| * | Add test_ignored_items_reportedAxel Aguado2023-01-211-0/+16
|/ /
* | Merge pull request #1541 from hugovk/add-3.11Sebastian Thiel2023-01-177-30/+44
|\ \ | | | | | | Declare support for Python 3.11
| * | Lint with Flake8 via pre-commitHugo van Kemenade2023-01-166-27/+40
| | |
| * | Upgrade Python syntax with pyupgrade --py37-plusHugo van Kemenade2023-01-151-1/+1
| | |
| * | Fix typoHugo van Kemenade2023-01-151-2/+2
| | |
| * | Declare support for Python 3.11Hugo van Kemenade2023-01-151-0/+1
|/ /
* | Merge pull request #1537 from teknoraver/mainSebastian Thiel2023-01-142-2/+33
|\ \ | | | | | | fix files list on file rename
| * | fix files list on file renameMatteo Croce2023-01-132-2/+33
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | GitPython parses the output of `git diff --numstat` to get the files changed in a commit. This breaks when a commit contains a file rename, because the output of `git diff` is different than expected. This is the output of a normal commit: $ git diff --numstat 8f41a390bf9a^ 8f41a390bf9a 30 5 test/test_repo.py And this a commit containing a rename: $ git diff --numstat 185d847ec764^ 185d847ec764 3 1 .github/workflows/{test_pytest.yml => Future.yml} This can be triggered by this code: for commit in repo.iter_commits(): print(commit.hexsha) for file in commit.stats.files: print(file) Which will print for the normal commit: 8f41a390bf9a54db6f85032bc56b453307b95451 'test/test_repo.py' And when there is a rename: 185d847ec7647fd2642a82d9205fb3d07ea71715 '.github/workflows/{test_pytest.yml => Future.yml}' Additionally, when a path member is removed, the file list become a list of strings, breaking even more the caller. This is in the Linux kernel tree: $ git diff --numstat db401875f438^ db401875f438 1 1 tools/testing/selftests/drivers/net/mlxsw/{spectrum-2 => }/devlink_trap_tunnel_ipip6.sh and GitPython parses it as: db401875f438168c5804b295b93a28c7730bb57a ('tools/testing/selftests/drivers/net/mlxsw/{spectrum-2 => ' '}/devlink_trap_tunnel_ipip6.sh') Fix this by pasing the --no-renames option to `git diff` which ignores renames and print the same output as if the file was deleted from the old path and created in the new one: $ git diff --numstat --no-renames 185d847ec764^ 185d847ec764 57 0 .github/workflows/Future.yml 0 55 .github/workflows/test_pytest.yml
* | Merge pull request #1532 from marlamb/feature/reduce-resource-leaksSebastian Thiel2023-01-092-3/+9
|\ \ | | | | | | Fix some resource leaks by open file handles
| * | Fix some resource leaks by open file handlesMartin Lambertsen2023-01-092-3/+9
|/ /
* | Merge pull request #1531 from obfusk/fix-tempfileSebastian Thiel2023-01-083-454/+517
|\ \ | | | | | | use tempfile.TemporaryDirectory & fix clone_from_unsafe_protocol tests
| * | fix clone_from_unsafe_protocol testsFC Stegerman2023-01-081-5/+30
| | |
| * | replace tempfile.mkdtemp w/ tempfile.TemporaryDirectoryFC Stegerman2023-01-083-450/+488
| |/
* | Merge pull request #1530 from obfusk/fix-unsafe-param-docstringSebastian Thiel2023-01-083-3/+20
|\ \ | |/ |/| fix/add allow_unsafe_* params in docstrings + fix typo
| * fix/add allow_unsafe_* params in docstrings + fix typoFC Stegerman2023-01-083-3/+20
|/
* tests: Use `command -v` instead of third-party `which` programMichał Górny2022-12-311-1/+1
| | | | | | | | Use `command -v` to locate the git executable instead of `which`. The former is guaranteed to always be available according to POSIX, while which(1) is a redundant third-party tool that is slowly being phased out from Linux distributions. In particular, Gentoo no longer installs it by default.
* Fix Sphinx rendering errorsStephan Creutz2022-12-2912-17/+37
| | | | | | | | These errors are mostly fixed by either adding blank lines or single spaces for Sphinx documentation key words. The commit solely includes documentation changes, no functional changes.
* adjust changelog prior to release3.1.30Sebastian Thiel2022-12-292-1/+6
|