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: bazel-contrib/bazel-gazelle
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v0.41.0
Choose a base ref
...
head repository: bazel-contrib/bazel-gazelle
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v0.42.0
Choose a head ref
  • 8 commits
  • 24 files changed
  • 7 contributors

Commits on Jan 8, 2025

  1. Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    3315c48 View commit details

Commits on Jan 10, 2025

  1. Add docs for go_deps extension (#2010)

    **What type of PR is this?**
    
    Documentation
    
    **What package or component does this PR mostly affect?**
    
    go_deps
    
    **What does this PR do? Why is it needed?**
    
    Document the `go_deps` extension.
    
    Also remove some ancient and unsupported repo rules from the docs.
    
    **Which issues(s) does this PR fix?**
    
    Fixes #
    
    **Other notes for review**
    
    ---------
    
    Co-authored-by: Jay Conrod <[email protected]>
    fmeum and jayconrod authored Jan 10, 2025
    Copy the full SHA
    3cebbd7 View commit details
  2. Get gazelle_test and generationtest to pass with Bzlmod (#2009)

    Also clean up `.bazelrc` and presubmit configs.
    fmeum authored Jan 10, 2025
    Copy the full SHA
    7f7af3b View commit details

Commits on Jan 14, 2025

  1. fix: use REPO.bazel and MODULE.bazel when finding repo root (#2013)

    Fixes #2012
    alexeagle authored Jan 14, 2025
    Copy the full SHA
    d172a22 View commit details

Commits on Jan 16, 2025

  1. merger: handle ident load (#2014)

    **What type of PR is this?**
    Bug fix
    
    **What package or component does this PR mostly affect?**
    merger
    
    **What does this PR do? Why is it needed?**
    I found that used idents in assignment expression rhs position have
    their load statements removed. This PR adds those idents to used symbols
    collection in `FixLoads` fn.
    chasestarr authored Jan 16, 2025
    Copy the full SHA
    edfa984 View commit details

Commits on Jan 17, 2025

  1. Fix version resolution with bazel_deps and improve warnings (#2015)

    Drops a stray `continue` that could result in incorrect Go module
    versions being reported if they are declared by a module further down in
    the tree than one that declares a `bazel_dep`.
    
    Warnings now look like this:
    ```
    Mismatch between versions requested for Go module github.com/cloudflare/circl:
    
      bazel_dep version (MODULE.bazel): 1.3.7 (as "circl")
      Go module version (go.mod):       1.3.8
    
    Either ensure that you have
    
      bazel_dep(module_name = "circl", version = "1.3.8")
    
    in your MODULE.bazel file or downgrade the Go module version via
    
      bazel run @my_rules_go//go:go -- get github.com/cloudflare/[email protected]
    
    ...
    The following Go modules were required by the root module at the given versions, but were implicitly updated to higher versions due to transitive dependencies:
    
      github.com/stretchr/testify: v1.6.1 -> v1.8.0
      github.com/bazelbuild/buildtools: v0.0.0-20230317132445-9c3c1fc0106e -> v0.0.0-20240918101019-be1c24cc9a44
      github.com/bazelbuild/rules_go: v0.39.1 -> v0.50.1
      golang.org/x/sys: v0.15.0 -> v0.25.0
    
    Update the root module's dependencies to match the resolved versions via:
    
      bazel run @my_rules_go//go:go -- get github.com/stretchr/[email protected] github.com/bazelbuild/[email protected] github.com/bazelbuild/[email protected] golang.org/x/[email protected]
    
    ```
    fmeum authored Jan 17, 2025
    Copy the full SHA
    1ad7ece View commit details

Commits on Jan 22, 2025

  1. fix(cmd/gazelle): small cleanups to fix-update (#2017)

    There are some small changes to fix update:
    - remove some legacy code that we don't need anymore
    - error handling simplification
    - unnecessary loops
    tyler-french authored Jan 22, 2025
    Copy the full SHA
    1bbedea View commit details

Commits on Jan 24, 2025

  1. fix: Relaxing label parsing for Bazel 8 (#2021)

    <!-- Thanks for sending a PR! Before submitting:
    
    1. If this is your first PR, please read CONTRIBUTING.md and sign the
    CLA
       first. We cannot review code without a signed CLA.
    2. Please file an issue *first*. All features and most bug fixes should
    have
    an associated issue with a design discussed and decided upon. Small bug
       fixes and documentation improvements don't need issues.
    3. New features and bug fixes must have tests. Documentation may need to
    be updated. If you're unsure what to update, send the PR, and we'll
    discuss
       in review.
    -->
    
    **What type of PR is this?**
    
    > Uncomment one line below and remove others.
    > Bug fix
    
    **What package or component does this PR mostly affect?**
    
    > For example:
    > all
    
    **What does this PR do? Why is it needed?**
    The label format changed with Bazel 8 and needs to be adapted here
    
    **Which issues(s) does this PR fix?**
    
    Fixes #2019
    
    **Other notes for review**
    darkrift authored Jan 24, 2025
    Copy the full SHA
    6699394 View commit details
Loading