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: kisielk/errcheck
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v1.6.0
Choose a base ref
...
head repository: kisielk/errcheck
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v1.6.1
Choose a head ref
  • 17 commits
  • 21 files changed
  • 6 contributors

Commits on Mar 28, 2021

  1. Provide go/analysis analyzer instance

    @dtcaciuc
    dtcaciuc committed Mar 28, 2021
    Configuration menu
    Copy the full SHA
    b5c4dc1 View commit details
    Browse the repository at this point in the history

  2. Fix tests

    @dtcaciuc
    dtcaciuc committed Mar 28, 2021
    Configuration menu
    Copy the full SHA
    83b278b View commit details
    Browse the repository at this point in the history

  3. analyzer: Return result

    @dtcaciuc
    dtcaciuc committed Mar 28, 2021
    Configuration menu
    Copy the full SHA
    7154fcc View commit details
    Browse the repository at this point in the history

Commits on Mar 29, 2021

  1. Add analyzer tests

    @dtcaciuc
    dtcaciuc committed Mar 29, 2021
    Configuration menu
    Copy the full SHA
    9b8014a View commit details
    Browse the repository at this point in the history

  2. Remove a TODO

    @dtcaciuc
    dtcaciuc committed Mar 29, 2021
    Configuration menu
    Copy the full SHA
    35d2ca6 View commit details
    Browse the repository at this point in the history

Commits on May 12, 2021

  1. Exclude (*Pipe{Read,Write}r).CloseWithError

    @jcharum
    jcharum committed May 12, 2021
    Configuration menu
    Copy the full SHA
    ba72820 View commit details
    Browse the repository at this point in the history

Commits on May 20, 2021

  1. Merge pull request #200 from jcharum/io-excludes 

    Exclude (*io.Pipe{Read,Write}r).CloseWithError
    @kisielk
    kisielk authored May 20, 2021
    Configuration menu
    Copy the full SHA
    98b1bd1 View commit details
    Browse the repository at this point in the history

Commits on May 28, 2021

  1. Merge master

    @dtcaciuc
    dtcaciuc committed May 28, 2021
    Configuration menu
    Copy the full SHA
    2cfce26 View commit details
    Browse the repository at this point in the history

  2. Merge pull request #198 from dtcaciuc/analyzer 

    Provide go/analysis analyzer instance
    @dtcaciuc
    dtcaciuc authored May 28, 2021
    Configuration menu
    Copy the full SHA
    cf5d9bf View commit details
    Browse the repository at this point in the history

Commits on Jun 24, 2021

  1. close file explicitly, to avoid "max open files" error

    @lance6716
    lance6716 authored Jun 24, 2021
    Configuration menu
    Copy the full SHA
    e6ab97f View commit details
    Browse the repository at this point in the history

Commits on Jun 25, 2021

  1. Merge pull request #202 from lance6716/patch-1 

    close file explicitly, to avoid "max open files" error
    @kisielk
    kisielk authored Jun 25, 2021
    Configuration menu
    Copy the full SHA
    8ddee48 View commit details
    Browse the repository at this point in the history

Commits on Dec 22, 2021

  1. Use "go install" instead of "go get" in README.md 

    See: https://go.dev/doc/go-get-install-deprecation
It's available since Go 1.16 and running "go get" recommends it in the future.
It the change is too new, maybe we can specify that "go get" should be used in older versions and "go install" in newer ones (since they allow you to specify a fixed version as well)
    @eliasdaler
    eliasdaler authored Dec 22, 2021
    Configuration menu
    Copy the full SHA
    37cf04d View commit details
    Browse the repository at this point in the history

  2. Merge pull request #204 from eliasdaler/patch-1 

    Use "go install" instead of "go get" in README.md
    @dtcaciuc
    dtcaciuc authored Dec 22, 2021
    Configuration menu
    Copy the full SHA
    b275da0 View commit details
    Browse the repository at this point in the history

Commits on Mar 16, 2022

  1. Test with Go 1.18, update deps

    @dtcaciuc
    dtcaciuc committed Mar 16, 2022
    Configuration menu
    Copy the full SHA
    a4918d3 View commit details
    Browse the repository at this point in the history

Commits on Mar 26, 2022

  1. Merge pull request #207 from dtcaciuc/go18 

    Test with Go 1.18, update deps
    @dtcaciuc
    dtcaciuc authored Mar 26, 2022
    Configuration menu
    Copy the full SHA
    e62617a View commit details
    Browse the repository at this point in the history

Commits on May 20, 2022

  1. Handle embedded nil interfaces. 

    Fixes #211
    @kisielk
    kisielk committed May 20, 2022
    Configuration menu
    Copy the full SHA
    ab13fbe View commit details
    Browse the repository at this point in the history

  2. Merge pull request #213 from kisielk/embedded-nil 

    Handle embedded nil interfaces.
    @kisielk
    kisielk authored May 20, 2022
    Configuration menu
    Copy the full SHA
    98d3f84 View commit details
    Browse the repository at this point in the history
Loading