-
Notifications
You must be signed in to change notification settings - Fork 825
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
Only install doc and lint dependencies (not a full install) #498
Labels
Comments
mayurkale22
added
feature-request
up-for-grabs
Good for taking. Extra help will be provided by maintainers
labels
Nov 6, 2019
I'll do this |
mayurkale22
removed
the
up-for-grabs
Good for taking. Extra help will be provided by maintainers
label
Nov 8, 2019
This was referenced Nov 8, 2019
Closed via #508 |
pichlermarc
pushed a commit
to dynatrace-oss-contrib/opentelemetry-js
that referenced
this issue
Dec 15, 2023
* feat: add browser extension * Update browser-extension/tsconfig.json Co-authored-by: Bartlomiej Obecny <[email protected]> * Update browser-extension/tsconfig.json Co-authored-by: Bartlomiej Obecny <[email protected]> * Update browser-extension/package.json Co-authored-by: Bartlomiej Obecny <[email protected]> * Update browser-extension/src/types.ts Co-authored-by: Bartlomiej Obecny <[email protected]> * Apply suggestions from code review Co-authored-by: Bartlomiej Obecny <[email protected]> * fix: move browser-extension to packages/opentelemetry-injection-browser-extension * fix: clean up browser extension and add react components * fix: implement suggestions for browser extension * fix: move extension into new folder * fix: name of browser extension in package.json * docs: update browser extension README.md and add images * fix: popup.png to console.png in browser extension README.md * fix: remove incomplete code for background span collector * fix: update dependencies and build script for browser extension * fix: make manifest version 2 the default for the autoinjection browser extension * feat: add permission management to browser extension * fix: remove unused npm scripts * fix: move api to peer dependency for browser extension * fix: add api to dev dependency for browser extension * fix: update versions for browser extension * fix: remove defect codecov script from browser extension * fix: pin api version for browser extension * feat: add tests to browser extension * fix: add nyc dependency to browser extension * fix: add ts-mocha dependency to browser extension * fix: add missing dependencies to browser extension * fix: downgrade dependencies for browser extension in node8 * fix: remove orphaned console.log from browser extension * fix: upgrade api dependency to 1.0 for browser ext * fix: revert upgrade of api dependency for browser extension * fix: upgrade dependency to api 1.0 for browser extension * Update packages/opentelemetry-browser-extension-autoinjection/package.json Co-authored-by: Daniel Dyla <[email protected]> * fix: upgrade dependencies for browser extension * fix: add npm run codecov to browser extension * fix: add codecov dependency to browser extension * fix: increase code coverage for browser ext instrumentation test Co-authored-by: Bartlomiej Obecny <[email protected]> Co-authored-by: Daniel Dyla <[email protected]> Co-authored-by: Valentin Marchaud <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Original issue #481
Full dependencies installation takes > 3 mins. Consider to install only docs and lint related dependencies for lint_&_docs job. This will make build little more faster.
The text was updated successfully, but these errors were encountered: