Skip to content

Only run npm if required#5363

Merged
radical merged 1 commit intomainfrom
sebros/npmbuild
Aug 20, 2024
Merged

Only run npm if required#5363
radical merged 1 commit intomainfrom
sebros/npmbuild

Conversation

@sebastienros
Copy link
Copy Markdown
Contributor

@sebastienros sebastienros commented Aug 20, 2024

Fixes #5292

Microsoft Reviewers: Open in CodeFlow

@ghost ghost added the area-codeflow for labeling automated codeflow. intentionally a different color! label Aug 20, 2024
@sebastienros sebastienros changed the title Only run npm when required Only run npm if required Aug 20, 2024
@radical radical added area-engineering-systems infrastructure helix infra engineering repo stuff and removed area-codeflow for labeling automated codeflow. intentionally a different color! labels Aug 20, 2024
@radical
Copy link
Copy Markdown
Member

radical commented Aug 20, 2024

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@radical radical merged commit bad0a6c into main Aug 20, 2024
@radical radical deleted the sebros/npmbuild branch August 20, 2024 23:20
@sebastienros
Copy link
Copy Markdown
Contributor Author

@radical was kind of waiting for @JamesNK to tell us why we can't do that ;) We can still revert if there is a valid reason.

@radical
Copy link
Copy Markdown
Member

radical commented Aug 20, 2024

Oh, I'm so sorry!! Yes, we can revert it if required!

@github-actions github-actions bot locked and limited conversation to collaborators Sep 20, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-engineering-systems infrastructure helix infra engineering repo stuff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Local build should not require npm

2 participants