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

feat(): upgrade to angular@latest+rc #1580

Merged
merged 2 commits into from
Dec 18, 2019
Merged

feat(): upgrade to angular@latest+rc #1580

merged 2 commits into from
Dec 18, 2019

Conversation

emoralesb05
Copy link
Contributor

Description

Upgrade to latest 9+RC.

Test Steps

  • npm ci
  • npm run serve

General Tests for Every PR

  • npm run serve:prod still works.
  • npm run tslint passes.
  • npm run stylelint passes.
  • npm test passes and code coverage is not lower.
  • npm run build:lib still works.

@@ -38,7 +38,7 @@ elif [ "${MODE}" = "release" ]; then
elif [ "${MODE}" = "unit-test" ]; then
npm run test
elif [ "${MODE}" = "audit" ]; then
npm audit
npm audit --only=prod --audit-level=high
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we also add the --production flag? I believe this will make it so it ignores devDependencies, based on this:
npm/cli#202

@jeremysmartt jeremysmartt merged commit e46ee30 into develop Dec 18, 2019
@jeremysmartt jeremysmartt deleted the feat/upgrade-ng branch December 18, 2019 17:56
kriswinbush pushed a commit to kriswinbush/covalent that referenced this pull request Feb 20, 2020
* feat(): upgrade to angular@latest+rc

* fix(): npm audit restriction
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