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

load Assignees for mention #1

Conversation

a1012112796
Copy link

don't forgot load Assignees, or can't mention because of empty tributeValues

ref: https://github.com/go-gitea/gitea/blob/dc5f2cf5906ec2f87ad47ea4724cc245c401eef6/templates/base/head_script.tmpl#L27-L29

Signed-off-by: a1012112796 <1012112796@qq.com>
Copy link
Owner

@delvh delvh left a comment

Choose a reason for hiding this comment

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

Looks fine.
Unfortunately, I won't be able to test it today.
Will merge it as soon as I've been able to test it.

@delvh
Copy link
Owner

delvh commented Jan 31, 2023

Sorry, I completely forgot about this PR.
Have just been able to test that it works as intended.

@delvh delvh merged this pull request into delvh:feature/mentioning-on-release-description Jan 31, 2023
delvh pushed a commit that referenced this pull request May 23, 2024
Result of `go get -u golang.org/x/net; make tidy`.

This is related to the following vulncheck warning:
```
There are 2 vulnerabilities in modules that you require that are
neither imported nor called. You may not need to take any action.
See https://pkg.go.dev/golang.org/x/vuln/cmd/govulncheck for details.

Vulnerability #1: GO-2024-2687
    HTTP/2 CONTINUATION flood in net/http
  More info: https://pkg.go.dev/vuln/GO-2024-2687
  Module: golang.org/x/net
    Found in: golang.org/x/net@v0.22.0
    Fixed in: golang.org/x/net@v0.23.0

Vulnerability #2: GO-2022-0470
    No access control in github.com/blevesearch/bleve and bleve/v2
  More info: https://pkg.go.dev/vuln/GO-2022-0470
  Module: github.com/blevesearch/bleve/v2
    Found in: github.com/blevesearch/bleve/v2@v2.3.10
    Fixed in: N/A
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants