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.

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

chore: Improve npm install debug logging #652

Merged
merged 1 commit into from
May 13, 2024
Merged

chore: Improve npm install debug logging #652

merged 1 commit into from
May 13, 2024

Conversation

mildaniel
Copy link
Contributor

Issue #, if available:
#635

Description of changes:

  • Adds logging to make it explicit that sam build only install production npm dependencies.
  • Updates linting configuration to remove deprecated features

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@mildaniel mildaniel requested a review from a team as a code owner May 13, 2024 21:02
@mildaniel mildaniel requested review from mndeveci and lucashuy May 13, 2024 21:02
@mildaniel mildaniel enabled auto-merge May 13, 2024 21:07
@mildaniel mildaniel added this pull request to the merge queue May 13, 2024
Merged via the queue into aws:develop with commit 002b6f9 May 13, 2024
117 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mndeveci mndeveci mndeveci approved these changes

@sriram-mv sriram-mv sriram-mv approved these changes

@lucashuy lucashuy Awaiting requested review from lucashuy lucashuy was automatically assigned from aws/serverless-application-experience-sbt

Assignees
No one assigned
Labels
area/workflow/node_npm pr/internal
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mildaniel @sriram-mv @mndeveci