Skip to content

Update tool dependencies#36445

Merged
silverwind merged 4 commits intogo-gitea:mainfrom
silverwind:tooldeps-10
Jan 24, 2026
Merged

Update tool dependencies#36445
silverwind merged 4 commits intogo-gitea:mainfrom
silverwind:tooldeps-10

Conversation

@silverwind
Copy link
Copy Markdown
Member

Updates all dependencies in Makefile. The go fix was done automatically, I just altered the variable name.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jan 24, 2026
@github-actions github-actions bot added modifies/go Pull requests that update Go code modifies/internal labels Jan 24, 2026
@silverwind silverwind changed the title Tooldeps 10 Update tool dependencies Jan 24, 2026
@silverwind silverwind added the skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. label Jan 24, 2026
@silverwind silverwind requested a review from Copilot January 24, 2026 04:40
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Jan 24, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates developer/tooling dependencies in the Makefile and applies small Go “fix” refactors to use Cut helpers for clearer string/byte parsing.

Changes:

  • Bump tool versions in Makefile (golangci-lint, actionlint).
  • Refactor IPv6 hostport parsing in portOnly to use strings.Cut.
  • Refactor 4-byte aligned string parsing in NuGet PDB extraction to use bytes.Cut.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
modules/validation/binding.go Uses strings.Cut for simpler/clearer "]:" port extraction logic.
modules/packages/nuget/symbol_extractor.go Uses bytes.Cut to detect NUL-terminated strings while reading stream names.
Makefile Updates pinned versions for golangci-lint and actionlint.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Jan 24, 2026
@silverwind silverwind merged commit a608b9e into go-gitea:main Jan 24, 2026
24 checks passed
@GiteaBot GiteaBot added this to the 1.26.0 milestone Jan 24, 2026
@silverwind silverwind deleted the tooldeps-10 branch January 24, 2026 08:58
zjjhot added a commit to zjjhot/gitea that referenced this pull request Jan 26, 2026
* giteaofficial/main:
  Normalize guessed languages for code highlighting (go-gitea#36450)
  Add `knip` linter (go-gitea#36442)
  Fix various bugs (go-gitea#36446)
  Update tool dependencies (go-gitea#36445)
  Update JS dependencies, adjust webpack config, misc fixes (go-gitea#36431)
  fix: Improve image captcha contrast for dark mode (go-gitea#36265)
  Refactor template render (go-gitea#36438)
  Add documentation for markdown anchor post-processing (go-gitea#36443)
  Fix markup heading parsing, fix emphasis parsing (go-gitea#36284)
  Front port changelog for 1.25.4 (go-gitea#36432)
  Bugfix: Potential incorrect runID in run status update (go-gitea#36437)
  Restrict branch naming when new change matches with protection rules (go-gitea#36405)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/go Pull requests that update Go code modifies/internal skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants