Skip to content

fix: remove preinstall step#1422

Merged
aaudzei merged 2 commits into
mainfrom
users/antonaudzei/v2.8.1
Jul 14, 2026
Merged

fix: remove preinstall step#1422
aaudzei merged 2 commits into
mainfrom
users/antonaudzei/v2.8.1

Conversation

@aaudzei

@aaudzei aaudzei commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Remove script that maybe in conflict with enterprise or engineering team settings/env

As a backup there is mention of that step in troubleshooting guide:

npm config get registry

PR Checklist

  • I have read the contribution guidelines
  • I have read the code of conduct guidelines
  • Title of the pull request is clear and informative.
  • 👌 Code hygiene
  • 🔭 Telemetry added, updated, or N/A
  • 📄 Documentation added, updated, or N/A
  • 🛡️ Automated tests added, or N/A

🧪 How did you test it?

Replace with use cases tested and models used

@aaudzei aaudzei requested a review from a team as a code owner July 14, 2026 13:41
@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@aaudzei aaudzei merged commit 37716af into main Jul 14, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants