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

Use FormatInt instead of Sprint #17671

Closed
wants to merge 2 commits into from
Closed

Use FormatInt instead of Sprint #17671

wants to merge 2 commits into from

Conversation

Gusted
Copy link
Contributor

@Gusted Gusted commented Nov 16, 2021

@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Nov 16, 2021
@zeripath
Copy link
Contributor

It might be better to just fold this in to #17661

@Gusted
Copy link
Contributor Author

Gusted commented Nov 16, 2021

It might be better to just fold this in to #17661

I'm not sure if it's within the scope of that PR that is adding a feature.

@zeripath
Copy link
Contributor

So I think this might be a no-op as I've just fixed this in another PR

@Gusted Gusted closed this Nov 16, 2021
@Gusted Gusted deleted the use-strconv branch November 16, 2021 19:01
@go-gitea go-gitea locked and limited conversation to collaborators Apr 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@zeripath zeripath zeripath approved these changes

Assignees
No one assigned
Labels
lgtm/need 1 This PR needs approval from one additional maintainer to be merged.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Gusted @zeripath @GiteaBot