Skip to content
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

Backup package(-lock).json update with local paths with V1 support #252

Closed

Conversation

lkolacek
Copy link
Contributor

We might not need to support "deprecated" package-lock.json v1 for npm version 6,
but I'll backup this code changes in case we would need them.

Maintainers will complete the following section

  • Commit messages are descriptive enough
  • Code coverage from testing does not decrease and new code is covered
  • Docs updated (if applicable)
  • Docs links in the code are still valid (if docs were updated)

For V2+ package.json(s) update only https and git sources with empty string.
   in package-lock.json update git and https package dependencies with empty string.

STONEBLD-1108

Signed-off-by: Ladislav Kolacek <[email protected]>
Logic for fetching sources was moved to _resolve_npm
Add unit tests for methods:
* check_if_package_is_workspace
* replace_dependency

STONEBLD-1108

Signed-off-by: Ladislav Kolacek <[email protected]>
* Updated package.json files with local paths to packages were added to build-config.json
* Promote npm test scenarios (npm_smoketest_lockfile1, npm_smoketest_lockfile2, npm_smoketest_lockfile3) to end to end tests
* Add Containerfiles for new End to End tests
* Change type of expected_cmd_output for end to end tests to list

STONEBLD-1108

Signed-off-by: Ladislav Kolacek <[email protected]>
@lkolacek lkolacek marked this pull request as draft July 20, 2023 08:59
@lkolacek
Copy link
Contributor Author

lkolacek commented Oct 9, 2023

Closing this PR. V1 support for npm will not be included in Cachi2 project.

@lkolacek lkolacek closed this Oct 9, 2023
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.

1 participant