Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Improve log format#33814

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
wxiaoguang merged 3 commits intogo-gitea:mainfromwxiaoguang:fix-log
Mar 8, 2025
Merged

Improve log format#33814

wxiaoguang merged 3 commits intogo-gitea:mainfromwxiaoguang:fix-log
Mar 8, 2025

Conversation

@wxiaoguang
Copy link
Contributor

@wxiaoguangwxiaoguang commentedMar 6, 2025
edited
Loading

Old:

2025/03/06 22:21:06 ...er/issues/indexer.go:76:func1() [I] PID 88713: Initializing Issue Indexer: bleve2025/03/06 22:21:12 ...eb/routing/logger.go:102:func1() [I] router: completed GET / for [::1]:52693, 200 OK in 15.9ms @ web/home.go:32(web.Home)

New:

2025/03/06 22:20:20 .../indexer/issues/indexer.go:76:InitIssueIndexer.1() [I] PID 88595: Initializing Issue Indexer: bleve2025/03/06 22:20:35 HTTPRequest [I] router: completed GET / for [::1]:52631, 200 OK in 7.5ms @ web/home.go:32(web.Home)

⚠️ BREAKING⚠️

Actually it isn't really breaking for most users.

If some users use some filters (regexp) to parse the log output, please make the filters work with the new log format.

TheFox0x7 and POPSuL reacted with heart emoji
@GiteaBotGiteaBot added the lgtm/need 2This PR needs two approvals by maintainers to be considered for merging. labelMar 6, 2025
@github-actionsgithub-actionsbot added the modifies/goPull requests that update Go code labelMar 6, 2025
@wxiaoguangwxiaoguang added the type/enhancementAn improvement of existing functionality labelMar 6, 2025
@wxiaoguangwxiaoguangforce-pushed thefix-log branch 2 times, most recently from9d7f782 tof7670beCompareMarch 6, 2025 14:32
@GiteaBotGiteaBot added lgtm/need 1This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2This PR needs two approvals by maintainers to be considered for merging. labelsMar 6, 2025
@GiteaBotGiteaBot added lgtm/doneThis PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1This PR needs approval from one additional maintainer to be merged. labelsMar 8, 2025
@wxiaoguangwxiaoguang merged commit4ed71eb intogo-gitea:mainMar 8, 2025
26 checks passed
@GiteaBotGiteaBot added this to the1.24.0 milestoneMar 8, 2025
@wxiaoguangwxiaoguang deleted the fix-log branchMarch 8, 2025 13:47
@wxiaoguangwxiaoguang added the pr/breakingMerging this PR means builds will break. Needs a description what exactly breaks, and how to fix it! labelMar 8, 2025
zjjhot added a commit to zjjhot/gitea that referenced this pull requestMar 10, 2025
* giteaofficial/main:  Move notifywatch to service layer (go-gitea#33825)  [skip ci] Updated translations via Crowdin  Only keep popular licenses (go-gitea#33832)  Removing unwanted ui container (go-gitea#33833)  Full-file syntax highlighting for diff pages (go-gitea#33766)  Improve theme display (go-gitea#30671)  Decouple context from repository related structs (go-gitea#33823)  Improve log format (go-gitea#33814)  Decouple diff stats query from actual diffing (go-gitea#33810)  Add global lock for migrations to make upgrade more safe with multiple replications (go-gitea#33706)  Do not show passkey on http sites (go-gitea#33820)
hiifong pushed a commit to hiifong/gitea that referenced this pull requestMar 10, 2025
@go-giteago-gitea locked asresolvedand limited conversation to collaboratorsJun 7, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

@lunnylunnylunny approved these changes

+1 more reviewer

@hiifonghiifonghiifong approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

lgtm/doneThis PR has enough approvals to get merged. There are no important open reservations anymore.modifies/goPull requests that update Go codepr/breakingMerging this PR means builds will break. Needs a description what exactly breaks, and how to fix it!type/enhancementAn improvement of existing functionality

Projects

None yet

Milestone

1.24.0

Development

Successfully merging this pull request may close these issues.

4 participants

@wxiaoguang@lunny@hiifong@GiteaBot

[8]ページ先頭

©2009-2025 Movatter.jp