Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: gcmarx/GitPython
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: f3ab5d3
Choose a base ref
...
head repository: gitpython-developers/GitPython
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 751cb2d
Choose a head ref
  • 12 commits
  • 11 files changed
  • 3 contributors

Commits on May 22, 2025

  1. Configuration menu
    Copy the full SHA
    8dc7552 View commit details
    Browse the repository at this point in the history

Commits on May 23, 2025

  1. Merge pull request gitpython-developers#2028 from gcmarx/remove-type-…

    …assertions
    
    remove type assertions from util.py
    Byron authored May 23, 2025
    Configuration menu
    Copy the full SHA
    bf51609 View commit details
    Browse the repository at this point in the history

Commits on May 28, 2025

  1. Merge pull request gitpython-developers#2026 from gcmarx/main

    correctly handle `uname-cmd` that doesn't point to an executable file
    EliahKagan authored May 28, 2025
    Configuration menu
    Copy the full SHA
    8576534 View commit details
    Browse the repository at this point in the history

Commits on May 30, 2025

  1. Use newer ruff style

    This updates `ruff` in `.pre-commit-config.yaml` from 0.6.0 to
    0.11.12, changes its `id` from the legacy `ruff` alias to
    `ruff-check` (which is better distinguished from `ruff-format`,
    which we also have a hook for), and applies the few style changes
    it newly recommends throughout the code. The style changes seem to
    make things slightly clearer overall.
    
    This also updates some other pre-commit hooks, but those don't
    require any changes to the code.
    
    Currently the `ruff` dependency in `requirements-dev.txt` doesn't
    specify a version, so no change is needed there. This update may
    be seen as bringing the `pre-commit` version in line with what
    users will usually have locally with `pip install -e ".[test]"`.
    
    The `pre-commit` hooks are how linting is currently done on CI, so
    this is updating `ruff` for CI. That's the most significant effect
    of this change. (`pre-commit` is run for linting on CI probably
    much more often than it is used locally, to manage pre-commit
    hooks or otherwise, in GitPython development.)
    EliahKagan committed May 30, 2025
    Configuration menu
    Copy the full SHA
    b7ce712 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    2d662cc View commit details
    Browse the repository at this point in the history
  3. Temporarily remove CodeQL workflow file

    So GitHub can regenerate a fresh new one based on current defaults.
    EliahKagan committed May 30, 2025
    Configuration menu
    Copy the full SHA
    0c0fc7e View commit details
    Browse the repository at this point in the history
  4. Recreate codeq.yml with current defaults

    This adds CodeQL scanning of GitHub Actions, while continuing to
    scan Python as well.
    
    This will subsequently be customized slightly to restore some
    elements of the preivous custom workflow that we may prefer.
    EliahKagan committed May 30, 2025
    Configuration menu
    Copy the full SHA
    847ead6 View commit details
    Browse the repository at this point in the history
  5. Keep running CodeQL on all branches, etc.

    This restores three aspects of the previous `codeql.yml`:
    
    - Run it on all branches, not just `main`.
    
    - Run it on the previous schedule rather than the new one, since
      there's no reason to change the schedule (though there's no
      reason to be attached to the old schedule either).
    
    - Use "CodeQL" rather than "CodeQL Advanced" as the workflow
      `name`, since this takes up less horizontal space when reading
      the reports from the checks.
    
    Of these, only the first is really significant.
    EliahKagan committed May 30, 2025
    Configuration menu
    Copy the full SHA
    d7ce09f View commit details
    Browse the repository at this point in the history
  6. Remove unnecessary permissions from codeql.yml

    This is another change back to the way we had it before, but the
    removals are based specifically on the guidance in the default
    workflow comments about why each permission was given by default.
    EliahKagan committed May 30, 2025
    Configuration menu
    Copy the full SHA
    89dbd4a View commit details
    Browse the repository at this point in the history
  7. Merge pull request gitpython-developers#2032 from EliahKagan/codeql-gha

    Have CodeQL scan GitHub Actions workflows as well as Python code
    EliahKagan authored May 30, 2025
    Configuration menu
    Copy the full SHA
    2e4a0d2 View commit details
    Browse the repository at this point in the history
  8. Specify explicit contents: read workflow permissions

    Three CI workflows that need only `contents: read` permissions and
    no other permissions did not have explicit permissions set, and
    would therefore be given default permissions configured for the
    repository, which might be more expansive than the workflows need.
    
    It is recommended to set explicit workflow permissions [1]. This
    does that, specifying permissions as `pythonpackage.yml` already
    did, and closing three `actions/missing-workflow-permissions`
    CodeQL alerts (new since gitpython-developers#2032 enabled scanning of GHA workflows).
    
    [1]: https://codeql.github.com/codeql-query-help/actions/actions-missing-workflow-permissions/
    EliahKagan committed May 30, 2025
    Configuration menu
    Copy the full SHA
    a9833d6 View commit details
    Browse the repository at this point in the history
  9. Merge pull request gitpython-developers#2033 from EliahKagan/gha-perm…

    …issions
    
    Specify explicit `contents: read` workflow permissions
    EliahKagan authored May 30, 2025
    Configuration menu
    Copy the full SHA
    751cb2d View commit details
    Browse the repository at this point in the history
Loading