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

Release v6.0.0 Stable #621

Closed
22 tasks done
malept opened this issue Nov 7, 2018 · 44 comments
Closed
22 tasks done

Release v6.0.0 Stable #621

malept opened this issue Nov 7, 2018 · 44 comments
Assignees

Comments

@malept
Copy link
Member

malept commented Nov 7, 2018

  • I have read the contribution documentation for this project.
  • I agree to follow the code of conduct that this project follows, as appropriate.
  • I have searched the issue tracker for an issue that matches the one I want to file, without success.

I'd like to have a solid plan for getting v6 out the door.

TODO

In no particular order.

@MarshallOfSound
Copy link
Member

MarshallOfSound commented Nov 8, 2018

Figure out what to do with the electron-forge NPM package

npm deprecate all versions of it with a message to move to @electron-forge/cli? Maybe a link to a docs page on how to move and use the importer described above ^^

@malept
Copy link
Member Author

malept commented Nov 9, 2018

Do we no longer want to use the electron-forge NPM package?

@MarshallOfSound
Copy link
Member

@malept I don't think so, we could alias it to @electron-forge/cli but I think we should just make people move 👍

@malept
Copy link
Member Author

malept commented Nov 17, 2018

I don't know, there's something to be said for having install instructions be npx electron-forge init my-electron-app and that's it. But I guess we live in a copy/paste world, and npx @electron-forge/cli init my-electron-app is about the same.

@MarshallOfSound
Copy link
Member

@malept We can have npx create-electron-app and yarn create electron-app for the one-liner no-scope commands

@malept
Copy link
Member Author

malept commented Dec 6, 2018

Update templates

From what we've talked about previously, we're going to reduce the templates to just a webpack template (and a parcel template perhaps). We should add guides to the docs for how to create apps for various popular frameworks, e.g. React via create-react-app plus electron-forge init --template=webpack. Alternatively, have docs for creating webpack-based Electron apps in general.

@philcockfield
Copy link

This issue suggests v6 stable release is imminent (??).

The documentation wisely says don't use the Webpack plugin yet...but given you're gearing up for stable release, is it possible to go ahead and work with Webpack?

Thanks so much for your work on "forge" - this is so very helpful.

@MarshallOfSound
Copy link
Member

@philcockfield Those warnings are just disclaimers at this point. A bunch of people are already using the webpack plugin, feel free to dive in

@malept
Copy link
Member Author

malept commented Jan 3, 2019

This issue suggests v6 stable release is imminent (??).

I would not characterize it as imminent. This issue is meant as a line in the sand to show progress towards releasing a stable version.

The documentation wisely says don't use the Webpack plugin yet...but given you're gearing up for stable release, is it possible to go ahead and work with Webpack?

What @MarshallOfSound said. There's not much in the way of documentation, so we're not yet comfortable recommending it yet.

@philcockfield
Copy link

Awesome - thanks.

@philcockfield
Copy link

Hey @MarshallOfSound - I know this is pre-release (or pre-docs and all), but I don't suppose you have a sample repo lying around anywhere of a webpack plugin configuration working?

I found this repo of yours https://github.com/MarshallOfSound/electron-forge-plugin-webpack/blob/master/example/react.js

But it's archived, and pretty old. Any hints would be awesome - thanks!

@philcockfield
Copy link

Also, for so I can understand the context of the new release - does @electron-forge/plugin-webpack supersede and make obsolete https://github.com/electron-userland/electron-webpack ?

@malept
Copy link
Member Author

malept commented Jan 11, 2019

electron-webpack is a different node module not affiliated with Electron Forge.

@philcockfield
Copy link

Oh! My bad, I thought that was under electron-userland banner.

Don't suppose there is working sample repo anywhere?

@malept
Copy link
Member Author

malept commented Jan 11, 2019

Based on https://github.com/search?q=%22electron-forge%22+%22plugin-webpack%22&type=Code there are people who are using that plugin, but I don't know how successful they are.

@ghost
Copy link

ghost commented Feb 13, 2019

It'd be nice to have more than 1 webpack (or parcel) template, where at least one acts like electron-compile. My app is still basically a webview into a local app, so I've only been focused on setting up the main process so far. I ended up using node-webpack-externals to basically ignore node_modules and package it as is (otherwise webpack tries to handle node-gyp files and other stuff). I also had to override the ignore to keep node_modules for main.

@malept
Copy link
Member Author

malept commented Feb 13, 2019

We don't plan on having more than one template per bundler included by default with Electron Forge v6. And as with the old 5.x templates, they are meant to be minimal starters (like electron-quick-start) and not boilerplates. (Quite frankly, it seems to be difficult to find someone to have the time to even do a minimal template.)

That is not to say that a community member can't contribute a third-party template.

@spasma
Copy link

spasma commented Feb 25, 2019

Can you make automation of new electron-prebuilt-compile packages also part of this release? electron-forge heavily depends on that package and it's pretty outdated.

Everybody is building apps on an old version of electron w/ security issues because of that. (or they hacked some things together to make it work)

@malept
Copy link
Member Author

malept commented Feb 25, 2019

As of Forge v6, electron-prebuilt-compile is no longer a requirement (although it's still somewhat supported through the optional electron-compile plugin). Its automation is specific to that module, though. See electron-userland/electron-prebuilt-compile#55 for the automation of releases there (which I've noticed you've already found).

@spasma
Copy link

spasma commented Feb 25, 2019

@malept ah I see, good to hear that. Thanks for the heads up!

@austencollins
Copy link

I love this project, great job! What's the status of V6?

@malept
Copy link
Member Author

malept commented Jun 13, 2019

I've updated the TODOs in the issue summary.

@austencollins
Copy link

Thanks. Looks close. Nice work!

@Bubbabry88
Copy link

Nice work

@damienallen
Copy link

I've had a great experience with electron-forge so far and really looking forward to v6!

Could anyone point me to documentation (couldn't find any) or summarize how a migration might look like? Just hoping to get a rough estimate of effort/time required.

@malept
Copy link
Member Author

malept commented Jul 16, 2019

It should mostly be config changes. I wrote a fairly simple importer from v5 to v6 here: https://github.com/electron-userland/electron-forge/blob/v6.0.0-beta.42/packages/api/core/src/util/upgrade-forge-config.ts (it's invoked by running electron-forge import, using v6, on a v5 app).

Obviously, there's also a cost to migrate off of electron-prebuilt-compile (and probably onto webpack), but that can be done in a separate step.

@miwelc
Copy link

miwelc commented Sep 10, 2019

Hi guys, do you have and ETA on this?
We need a more recent version of electron but the current dependency on electron-prebuilt-compile is limiting us (we use webpack so we don't really need it).
Thanks!

@malept
Copy link
Member Author

malept commented Sep 10, 2019

The main blocker is resolving bugs with webpack and native modules. We'd be happy to have help with this so we can (finally) release a stable version.

Aside from that, it's pretty stable.

@damienallen
Copy link

@malept The migration script worked great, thanks for that!

Overall, the move to version 6 (and from electron-compile to webpack) took me around 7 hours. A lot of time was also spent on getting TS up and running with webpack.

Awesome work on v6!

@deadcoder0904
Copy link
Contributor

Is v6 stable yet? Or usable?

@damienallen
Copy link

You can see from the checklist that only minor items remain. Very much usable and production ready in my view.

Feels great to ditch electron-compile!

@deadcoder0904
Copy link
Contributor

And are the docs at https://electronforge.io for v6? Because I vividly remember the docs were at https://v6.electronforge.io before

@MarshallOfSound
Copy link
Member

The v6 docs have been moved to the main domain. https://electronforge.io/ is now the docs site

@printjs

This comment has been minimized.

@malept

This comment has been minimized.

@b-zurg
Copy link
Contributor

b-zurg commented Apr 24, 2020

I get the impression that with just a little bit of focused attention this release could get out of beta. Is there anything I and others could do to help make this happen?

@malept
Copy link
Member Author

malept commented Apr 24, 2020

I get the impression that with just a little bit of focused attention this release could get out of beta. Is there anything I and others could do to help make this happen?

See #621 (comment)

@davidwinter
Copy link

Could the Snapcraft maker be resolved for a stable v6 version? It's currently impossible to build a snapcraft artefact on a standard version of Ubuntu.

It seems like there may be a PR that might resolve the issue: electron-userland/electron-installer-snap#75

And there are a number of issues open that appear related:

#1696
electron-userland/electron-installer-snap#83

I'd be very happy to help however I can, but I'm not sure asking people to run Forge from within a Docker container is ideal, considering snapcraft packages can be built easily when using electron-builder.

@mahnunchik
Copy link
Contributor

When is eta for 6.0.0?

@maccman
Copy link

maccman commented Feb 17, 2021

Can I help sponsor this?

@malept
Copy link
Member Author

malept commented Feb 17, 2021

@maccman I would be more than happy to have conversations about sponsoring work on Electron Forge. You can either use GitHub Sponsors or I'm on the official Electron Discord server.

@maccman
Copy link

maccman commented Feb 17, 2021

Boom - done! Thanks so much mate for working on this.

@Eli-Black-Work
Copy link

Is there any chance of releasing a new beta version? The previous beta release was 4 months ago.

In particular, I'm looking for the fix for #1377

No pressure; just checking 🙂

@MarshallOfSound
Copy link
Member

Ta Da? 🎉

dsanders11 pushed a commit that referenced this issue Jan 14, 2023
No functional changes. This is a minor refactor to avoid hardcoding
filenames or editorIds and to avoid hardcoding their lists. Instead,
generally iterate through the FILENAME_KEYS objects.

I started this cleanup while we were prototypeing a new 'test.js'
editor. It's less relevant since we discarded that idea, but it's still
preferable to use symbolic names instead of magic strings.
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

No branches or pull requests

16 participants