Skip to content

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.

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 5 commits into
base: main
Choose a base branch
from
Open

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

wants to merge 5 commits into from

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
@serhalp
Initial plan for fixing Verdaccio logs deprecation warning
d97c092 
Co-authored-by: serhalp <1377702+serhalp@users.noreply.github.com>
@serhalp
Fix Verdaccio deprecation warning by renaming logs to log
2d41514 
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
serhalp
serhalp requested changes Oct 23, 2025
View reviewed changes
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
@serhalp
Copy link
Member

serhalp commented Oct 23, 2025

hello @copilot ?
@serhalp
Revert unintended package-lock.json changes
c39a454 
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
@github-actions
Copy link

github-actions bot commented Oct 29, 2025

📊 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

Labels

None yet

2 participants

@serhalp