-
-
Notifications
You must be signed in to change notification settings - Fork 535
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
Use babel-preset-env in 'init' #47
Conversation
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.
In general it makes sense, and after much thought technically this isn't breaking so thats good too 👍
In general in this repo we use git-cz
for commit messages so that we can generate changelogs and versions automatically. And it looks cooler in the commit history 😆
Simply do npm run commit
and fill in the prompts 😄
envTarget[1].targets.electron = electronPrebuilt.version; | ||
} | ||
|
||
fs.writeFileSync(path.join(dir, '.compilerc'), JSON.stringify(content, null, 2), 'utf8'); |
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.
To be async
await fs.writeFile(...)
@@ -50,7 +50,7 @@ describe('forge-config', () => { | |||
}); | |||
}); | |||
|
|||
describe.only('require-search', () => { |
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.
rebasing on master should pull this in
Also for some strange reason the CI didn't run, adding another commit should trigger those though 👍 |
6b81148
to
77b14e7
Compare
await initStarter(dir, program.template ? undefined : program.lintstyle); | ||
await initNPM(dir, program.template ? undefined : program.lintstyle); |
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.
Was moving this call intentional?
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.
Yes, we have to have .compilerc laid down before we can edit it
This PR switches the default Babel preset to use babel-preset-env - this preset adjusts based on the current Electron version which Babel plugins to use. This PR also enables async/await and React by default, since these features are super common and don't have any overhead to enable when not in-use.
More unit tests, fix "Save Fiddle"
This PR switches the default Babel preset to use babel-preset-env - this preset adjusts based on the current Electron version which Babel plugins to use. This PR also enables async/await and React by default, since these features are super common and don't have any overhead to enable if not in-use.