Do not include Scenes dev dependencies in Traffic Stats - Githubissues

apache / trafficcontrol

Apache Traffic Control is an Open Source implementation of a Content Delivery Network

https://trafficcontrol.apache.org/

Apache License 2.0

1.02k stars 339 forks source link

Do not include Scenes dev dependencies in Traffic Stats #8010

Closed zrhoffman closed 1 month ago

zrhoffman commented 1 month ago

NPM dev dependencies were being included in Traffic Stats. #8010 ensures that dev dependencies are omitted.

## Which Traffic Control components are affected by this PR?

Traffic Stats
Build system

What is the best way to verify this PR?

build Traffic Stats RPM
Look at the files
Ensure it does not include packages like e2e in the node_modules folder

If this is a bugfix, which Traffic Control versions contained the bug?

8.0.1

PR submission checklist

[x] This PR has tests
[ ] This PR has documentation
[x] This PR has a CHANGELOG.md entry
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

ntheanh201 commented 1 month ago

LGTM, I can build Traffic Stats with TrafficControl Grafana Scenes