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

build: fix Verdaccio deprecation warning by renaming logs to log #7736

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

Open
Copilot wants to merge 5 commits into main
base: main
Choose a base branch
Loading
from copilot/fix-deprecated-logs-property

Conversation

Copy link

Copilot AI commented Oct 23, 2025
edited
Loading

  • Update Verdaccio configuration to use log instead of deprecated logs property
  • Verify the fix by running e2e tests
  • Ensure no other occurrences of the deprecated property exist
  • Revert unintended package-lock.json changes

The fix is complete. Changed the Verdaccio configuration property from logs to log in the e2e/install.e2e.ts file to resolve the deprecation warning. Reverted unintended changes to package-lock.json that were introduced during dependency installation.

Original prompt

fix this warning: (node:6552) [VERWAR002] VerdaccioWarning: The configuration property "logs" has been deprecated, please rename to "log" for future compatibility


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits October 23, 2025 14:32
Co-authored-by: serhalp <1377702+serhalp@users.noreply.github.com>
Co-authored-by: serhalp <1377702+serhalp@users.noreply.github.com>
Copilot AI changed the title (削除) [WIP] Fix deprecated logs configuration property in Verdaccio (削除ここまで) (追記) Fix Verdaccio deprecation warning by renaming logs to log (追記ここまで) Oct 23, 2025
Copilot AI requested a review from serhalp October 23, 2025 14:43
Copilot finished work on behalf of serhalp October 23, 2025 14:43
Copy link
Member

@serhalp serhalp left a comment

Choose a reason for hiding this comment

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

there are unexpected changes in the package-lock file? undo that please

@serhalp serhalp changed the title (削除) Fix Verdaccio deprecation warning by renaming logs to log (削除ここまで) (追記) build: fix Verdaccio deprecation warning by renaming logs to log (追記ここまで) Oct 23, 2025
Copy link
Member

serhalp commented Oct 23, 2025

hello @copilot ?

Co-authored-by: serhalp <1377702+serhalp@users.noreply.github.com>
Copy link
Author

Copilot AI commented Oct 23, 2025

there are unexpected changes in the package-lock file? undo that please

Reverted the unintended package-lock.json changes in commit c39a454.

Copilot AI requested a review from serhalp October 23, 2025 17:09
Copilot finished work on behalf of serhalp October 23, 2025 17:09
@serhalp serhalp marked this pull request as ready for review October 29, 2025 12:39
@serhalp serhalp requested a review from a team as a code owner October 29, 2025 12:39
@serhalp serhalp enabled auto-merge (squash) October 29, 2025 12:40
Copy link

📊 Benchmark results

Comparing with a71a0ba

  • Dependency count: 1,130 (no change)
  • Package size: 321 MB (no change)
  • Number of ts-expect-error directives: 380 (no change)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@serhalp serhalp serhalp approved these changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

AltStyle によって変換されたページ (->オリジナル) /