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.

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

feat(turbo): support extends on turbo.json and detecting turbo to prevent re-invocation #152

Merged
merged 2 commits into from
Jun 4, 2023

Conversation

fuxingloh
Copy link
Contributor

@fuxingloh fuxingloh commented May 31, 2023

What this PR does / why we need it:

@levaintech maintains an internal fork of sticky, this PR syncs the quality of life update that were introduced to turbo.

Additional comments?:

This PR contains breaking changes when sticky-turbo is used programmatically.

feat(sticky-jest-turbo): detect turbo is the current process to prevent re-invocation
chore(*): fix wrong lint script

This PR contains breaking changes when `sticky-turbo` is used programmatically.
@netlify
Copy link

netlify bot commented May 31, 2023

Deploy Preview for sticky-birthday ready!

Name Link
🔨 Latest commit 64f445e
🔍 Latest deploy log https://app.netlify.com/sites/sticky-birthday/deploys/6476c1597113f000083e429a
😎 Deploy Preview https://deploy-preview-152--sticky-birthday.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@codecov
Copy link

codecov bot commented May 31, 2023

Codecov Report

Merging #152 (64f445e) into main (de893cb) will increase coverage by 5.33%.
The diff coverage is 81.69%.

@@            Coverage Diff             @@
##             main     #152      +/-   ##
==========================================
+ Coverage   67.39%   72.72%   +5.33%     
==========================================
  Files           5        7       +2     
  Lines          92      176      +84     
  Branches        3       19      +16     
==========================================
+ Hits           62      128      +66     
- Misses         30       45      +15     
- Partials        0        3       +3     
Impacted Files Coverage Δ
packages/sticky-turbo/src/Turbo.ts 81.69% <81.69%> (ø)

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@thedoublejay
Copy link
Contributor

I think this PR might solve this #93

const paths = cwd.split('/');

for (let i = 0; i < depth; i += 1) {
const path = `${paths.join('/')}/turbo.json`;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah maybe not because we still hardcode it to "/"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let me fix this in another PR

@fuxingloh fuxingloh merged commit 71d60a1 into main Jun 4, 2023
12 checks passed
@fuxingloh fuxingloh deleted the fuxingloh/fork-update branch June 4, 2023 00:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature New feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants