-
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
ci: install minimal lint & doc deps #508
ci: install minimal lint & doc deps #508
Conversation
54babbc
to
b8def59
Compare
Overall time reduction from 4:28 on the last Full test run reduced from ~9 minutes to ~6 minutes |
Codecov Report
@@ Coverage Diff @@
## master #508 +/- ##
=========================================
+ Coverage 91.83% 92.73% +0.9%
=========================================
Files 115 135 +20
Lines 5656 6453 +797
Branches 521 568 +47
=========================================
+ Hits 5194 5984 +790
- Misses 462 469 +7
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
* feat: add responseHook config to redis instrumentation * fix: lint fix * feat: add responseHook config to redis instrumentation * fix: lint fix * chore(deps): update all non-major dependencies (open-telemetry#483) * chore: generalize the instrumentation file name (open-telemetry#479) * feat: add responseHook config to graphql instrumentation * Apply suggestions from code review Co-authored-by: Bartlomiej Obecny <[email protected]> * test: add a test for a responseHook that isn't a function * fix: lint fix Co-authored-by: WhiteSource Renovate <[email protected]> Co-authored-by: Rauno Viskus <[email protected]> Co-authored-by: Bartlomiej Obecny <[email protected]>
Which problem is this PR solving?
Short description of the changes
This installs only the bare minimum required to run
yarn run check
andyarn run docs
. The reason for the oddlerna exec
symlinking is becausegts
looks for rules packages in the local node modules, and does not look in the global directory.