| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
There was a problem hiding this comment.
We should probably also set min-release-age=7 in .npmrc and remove the @actions/* and actions/* Dependabot cooldown exclusions. This can happen in a separate PR though.
Sorry, something went wrong.
There was a problem hiding this comment.
The configuration and validation check correctly implement the stated portability safeguard without affecting production workflows.
Review effort: Balanced
Findings: None
Makes npm lockfiles registry-agnostic and prevents internal feed URLs from being committed.
Changes:
| File | Description |
|---|---|
| .npmrc | Enables registry-agnostic lockfile generation. |
| .github/workflows/pr-checks.yml | Detects internal feed URLs in package-lock.json. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
This PR adds a check to make sure that certain URLs that would create issues for some contributors don't end up in package-lock.json.
Risk assessment
For internal use only. Please select the risk level of this change:
Which use cases does this change impact?
How did/will you validate this change?
If something goes wrong after this change is released, what are the mitigation and rollback strategies?
How will you know if something goes wrong after this change is released?
Are there any special considerations for merging or releasing this change?
Merge / deployment checklist