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: add *.app to eslintignore #313

Merged
merged 2 commits into from
May 6, 2021
Merged

fix: add *.app to eslintignore #313

merged 2 commits into from
May 6, 2021

Conversation

WillieRuemmele
Copy link
Member

What does this PR do?

adds the **/aura/**/*.app entry to eslintignore to ignore an aura app

What issues does this PR fix or reference?

@W-9229019@
forcedotcom/cli#1004

@WillieRuemmele WillieRuemmele requested a review from a team as a code owner May 5, 2021 16:17
@brpowell brpowell merged commit 7e4ff4f into develop May 6, 2021
@brpowell brpowell deleted the wr/eslintApp2 branch May 6, 2021 14:55
xyc pushed a commit that referenced this pull request May 10, 2021
xyc pushed a commit that referenced this pull request May 10, 2021
* feat: added default jest config in new project template (#289)

* docs: add semvar info to publishing doc (#312)

* chore: remove codecov.io (#315)

* fix: add *.app to eslintignore (#313)

* feat(gitignore): add dotenv files (#314)

* add code folders to analytics projects  (#316)

* fix: update .nvmrc to match volta config

* feat: add code folders to analytics projects

This is aura/, classes/, and lwc/, plus the corresponding eslint configs.
Also adds the analytics vscode extension to the extension recommendations.

* build: upgrade dependencies (#317)

* build: update nyc in packages/templates
* build: update lerna
* build: update commitlint deps
* build: upgrade yeoman packages

Co-authored-by: Philippe Ozil <[email protected]>
Co-authored-by: Shelby Holden <[email protected]>
Co-authored-by: Willie Ruemmele <[email protected]>
Co-authored-by: Jonathan Wiesel <[email protected]>
Co-authored-by: Greg Smith <[email protected]>
Co-authored-by: Bryan Powell <[email protected]>
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