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

fix(lerna): lerna version doesn't know we are sharing eslintignore file #1629

Conversation

nstuyvesant
Copy link
Contributor

Updates

  • lerna version for some unknown reason messes with prettier. Because prettier's ignore is set in .vscode/settings.json (and points to eslint's ignore file), it tries to parse yarn.lock. This PR creates a prettier ignore file that is identical to the eslint ignore file so lerna knows not to parse yarn.lock.

@nstuyvesant nstuyvesant requested review from theiliad, Akshat55 and a team as code owners July 20, 2023 17:33
@nstuyvesant nstuyvesant requested review from zvonimirfras and removed request for a team July 20, 2023 17:33
@netlify
Copy link

netlify bot commented Jul 20, 2023

Deploy Preview for carbon-charts-core ready!

Name Link
🔨 Latest commit de04bcc
🔍 Latest deploy log https://app.netlify.com/sites/carbon-charts-core/deploys/64b96fe73b4d2500088d96c9
😎 Deploy Preview https://deploy-preview-1629--carbon-charts-core.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 configuration.

@netlify
Copy link

netlify bot commented Jul 20, 2023

Deploy Preview for carbon-charts-angular ready!

Name Link
🔨 Latest commit de04bcc
🔍 Latest deploy log https://app.netlify.com/sites/carbon-charts-angular/deploys/64b96fe7978c9200086eb4c5
😎 Deploy Preview https://deploy-preview-1629--carbon-charts-angular.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 configuration.

@netlify
Copy link

netlify bot commented Jul 20, 2023

Deploy Preview for carbon-charts-react ready!

Name Link
🔨 Latest commit de04bcc
🔍 Latest deploy log https://app.netlify.com/sites/carbon-charts-react/deploys/64b96fe7339d200008aa605e
😎 Deploy Preview https://deploy-preview-1629--carbon-charts-react.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 configuration.

Copy link
Member

@theiliad theiliad left a comment

Choose a reason for hiding this comment

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

LGTM

@theiliad theiliad merged commit a533241 into carbon-design-system:master Jul 20, 2023
1 check passed
@theiliad theiliad deleted the fix-lerna-so-it-knows-where-prettier-ignore-file-is branch July 20, 2023 20:15
@nstuyvesant nstuyvesant restored the fix-lerna-so-it-knows-where-prettier-ignore-file-is branch July 20, 2023 20:42
@nstuyvesant nstuyvesant deleted the fix-lerna-so-it-knows-where-prettier-ignore-file-is branch July 20, 2023 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants