Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

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
@fuxingloh fuxingloh deleted the fuxingloh/fork-update branch June 4, 2023 00:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/feature New feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants