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.

Sign up for GitHub

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

Jump to bottom

Update go.mod dependencies #23126

Merged
merged 7 commits into from
Feb 24, 2023
Merged

Update go.mod dependencies #23126

merged 7 commits into from
Feb 24, 2023

Conversation

zeripath
Copy link
Contributor

This PR does a bulk update of a lot of our go deps.

I have not included nektos/act and xorm for the following reasons:

  • Xorm updates can sometimes be complex and I'd rather do that in a separate PR
  • I think people more update with the actions code should double check that the latest nektos/act library works correctly.

@zeripath
update chroma
6aad58f 
Signed-off-by: Andrew Thornton <art27@cantab.net>
@zeripath
update golang.org/x/crypto
a083ad5 
Signed-off-by: Andrew Thornton <art27@cantab.net>
@zeripath
update up-to-humanize (includes mssql driver)
6d9b29f 
Signed-off-by: Andrew Thornton <art27@cantab.net>
@zeripath
update up to and including bluemonday
7c76cab 
Signed-off-by: Andrew Thornton <art27@cantab.net>
@zeripath
all others except nektos/act and xorm
98ec35b 
Signed-off-by: Andrew Thornton <art27@cantab.net>
@zeripath zeripath added this to the 1.20.0 milestone Feb 24, 2023
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Feb 24, 2023
@techknowlogick
Copy link
Member

techknowlogick commented Feb 24, 2023 •
edited
Loading

CI fail is related I guess some licenses may have changed. Those should be reviewed prior to merge otherwise LGTM

It was cli library that updated the date in license. I've just pushed the update license file.

@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 Feb 24, 2023
@zeripath zeripath merged commit 740a5ec into go-gitea:main Feb 24, 2023
@zeripath zeripath deleted the update-deps branch February 24, 2023 20:18
zjjhot added a commit to zjjhot/gitea that referenced this pull request Feb 27, 2023
@zjjhot
Merge remote-tracking branch 'giteaofficial/main'
aa04d18 
* giteaofficial/main:
  Revert "[skip ci] Updated translations via Crowdin" (go-gitea#23167)
  [skip ci] Updated translations via Crowdin
  Return 404 instead of 403 if user can not access the repo (go-gitea#23155)
  [skip ci] Updated licenses and gitignores
  Fix secrets overview page missing from docs sidebar (go-gitea#23143)
  Add "Reviewed by you" filter for pull requests (go-gitea#22927)
  Projects: rename Board to Column in interface and improve consistency (go-gitea#22767)
  Fix DBConsistency checks on MSSQL (go-gitea#23132)
  Add pagination for dashboard and user activity feeds (go-gitea#22937)
  Update go.mod dependencies (go-gitea#23126)
  Show empty repos in Admin Repository Management page (go-gitea#23114)
  Redirect to the commit page after applying patch (go-gitea#23056)
  Bump go.etcd.io/bbolt and blevesearch deps (go-gitea#23062)
  Fix height for sticky head on large screen on PR page (go-gitea#23111)
@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@delvh delvh delvh approved these changes

Assignees
No one assigned
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore.
Projects
None yet
Milestone
1.20.0
Development

Successfully merging this pull request may close these issues.
4 participants
@zeripath @techknowlogick @delvh @GiteaBot