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

build(deps): bump github.com/daixiang0/gci from 0.12.3 to 0.13.3 #4522

Merged
merged 5 commits into from
Mar 28, 2024

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Mar 17, 2024

Bumps github.com/daixiang0/gci from 0.12.3 to 0.13.3.
See full diff in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot added dependencies Relates to an upstream dependency go Pull requests that update Go code labels Mar 17, 2024
@ldez ldez added blocked Need's direct action from maintainer linter: update version Update version of linter labels Mar 17, 2024
@ldez
Copy link
Member

ldez commented Mar 17, 2024

Blocked because this will never work as expected with golangci-lint.

I created, a few months ago, https://github.com/golangci/modinfo to handle this kind of problem, I think that gci should use it.

Also, all the existing linters that need the module information use the same approach (it's also the same as https://github.com/golangci/modinfo) because it's the more reliable solution and recommended by the Go team.

Related to #4484

@CarlosRDomin
Copy link

How easy is the change? I could try to give it a go this week so we can get support for gci in golangci-lint, but could use some pointers of an example implementation / how to get started from @ldez 🙏🏼

@ldez
Copy link
Member

ldez commented Mar 19, 2024

How easy is the change?

The design of gci has been over-designed, so maybe or maybe not.

could use some pointers of an example implementation / how to get started

You can find examples inside the doc: https://github.com/golangci/modinfo?tab=readme-ov-file#examples

@dependabot dependabot bot force-pushed the dependabot/go_modules/github.com/daixiang0/gci-0.13.1 branch from 5b6408f to 0426420 Compare March 24, 2024 11:43
Bumps [github.com/daixiang0/gci](https://github.com/daixiang0/gci) from 0.12.3 to 0.13.1.
- [Release notes](https://github.com/daixiang0/gci/releases)
- [Commits](daixiang0/gci@v0.12.3...v0.13.1)

---
updated-dependencies:
- dependency-name: github.com/daixiang0/gci
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
@ldez ldez force-pushed the dependabot/go_modules/github.com/daixiang0/gci-0.13.1 branch from 0426420 to 27121d0 Compare March 27, 2024 01:24
@daixiang0
Copy link
Contributor

@ldez thanks for your nice help!

https://github.com/daixiang0/gci/releases/tag/v0.13.2 should fix it.

@ldez ldez changed the title build(deps): bump github.com/daixiang0/gci from 0.12.3 to 0.13.1 build(deps): bump github.com/daixiang0/gci from 0.12.3 to 0.13.2 Mar 27, 2024
@ldez ldez self-requested a review March 27, 2024 12:56
@ldez
Copy link
Member

ldez commented Mar 27, 2024

FYI v0.13.2 doesn't fix the problem, but it's not important because my current hack fixes the problem.

@CarlosRDomin
Copy link

I can confirm that your branch of golangci-lint works with the latest gci that supports localmodule, thank you so much @ldez 🎉

@golangci golangci deleted a comment from dependabot bot Mar 27, 2024
@ldez ldez changed the title build(deps): bump github.com/daixiang0/gci from 0.12.3 to 0.13.2 build(deps): bump github.com/daixiang0/gci from 0.12.3 to 0.13.3 Mar 28, 2024
@ldez ldez added this to the next milestone Mar 28, 2024
@ldez ldez removed the blocked Need's direct action from maintainer label Mar 28, 2024
@ldez ldez merged commit ed20557 into master Mar 28, 2024
13 checks passed
@ldez ldez deleted the dependabot/go_modules/github.com/daixiang0/gci-0.13.1 branch March 28, 2024 19:50
@ldez ldez modified the milestones: next, v1.58 May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Relates to an upstream dependency go Pull requests that update Go code linter: update version Update version of linter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants