Skip to content

build: Bump log normalizer to 2.0.0 #55990

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
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

build: Bump log normalizer to 2.0.0 #55990

wants to merge 2 commits into from
+2 −2

Conversation

@nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented Oct 24, 2025 •
edited
Loading

Prod Packages Operation Base Target
nextcloud/lognormalizer Upgraded v1.0.0 v2.0.0

@nickvergessen nickvergessen added this to the Nextcloud 33 milestone Oct 24, 2025
@nickvergessen nickvergessen self-assigned this Oct 24, 2025
@nickvergessen nickvergessen requested a review from a team as a code owner October 24, 2025 14:01
@nickvergessen nickvergessen requested review from Altahrim and removed request for a team October 24, 2025 14:01
@nickvergessen nickvergessen added the 3. to review Waiting for reviews label Oct 24, 2025
@nickvergessen nickvergessen changed the title Bump/lognormalizer build: Bump log normalizer to 2.0.0 Oct 24, 2025
@susnux
Copy link
Contributor

susnux commented Oct 27, 2025

Log spammed with:

├─ PHP Warning: Nextcloud\LogNormalizer\Normalizer::format(): Argument #1 ($data) must be passed by reference, value given in /var/www/html/lib/private/Log.php on line 161

susnux
susnux reviewed Oct 27, 2025
View reviewed changes
@nickvergessen
Copy link
Member Author

Log spammed with:

I had the composer.json commit commited instead of the auto commited code bump

Fixed now

@nickvergessen nickvergessen requested a review from susnux October 27, 2025 14:34
@nickvergessen
build: Bump log normalizer to 2.0.0
44b4b40 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
fix(log): Fix call to normalize function
2ec7fa8 
Signed-off-by: Joas Schilling <coding@schilljs.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@susnux susnux susnux approved these changes

@Altahrim Altahrim Altahrim approved these changes

@come-nc come-nc Awaiting requested review from come-nc

Assignees

@nickvergessen nickvergessen

Labels

3. to review Waiting for reviews

Projects

None yet

Milestone

Nextcloud 33

Development

Successfully merging this pull request may close these issues.

3 participants

@nickvergessen @susnux @Altahrim