Skip to content
This repository has been archived by the owner on Oct 2, 2020. It is now read-only.

retire electron-prebuilt #275

Merged
merged 3 commits into from
Aug 14, 2017
Merged

retire electron-prebuilt #275

merged 3 commits into from
Aug 14, 2017

Conversation

zeke
Copy link
Member

@zeke zeke commented Aug 8, 2017

The electron module will now be published to npm as part of the Electron release process, obviating the need for electron-prebuilt and the electron-prebuilt-updater Heroku app. This repo's history has been imported wholesale into electron/electron. See electron/electron#10172 for technical details.

This PR updates the README with info about the change and adds some backstory so people know what the role of this repo was in the past, and why we made the change.

We'll also sweep through the open issues and PRs here, migrating anything pertinent to electron/electron.

Copy link

@jkleinsc jkleinsc left a comment

Choose a reason for hiding this comment

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

Looks good to me except for the small grammatical correction needed.

README.md Outdated
and are incrementally documenting and [improving our release process](https://github.com/electron/electron/blob/master/docs/development/releasing.md).
As we've added support for things like
[TypeScript definitions](https://electron.atom.io/blog/2017/06/01/typescript),
it's been challening to work these additions into the `electron` -> `electron-prebuilt-updater` -> `electron-prebuilt` release flow.
Copy link

Choose a reason for hiding this comment

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

it's been challening should be it's been challenging

@vanessayuenn
Copy link
Contributor

I love the back story section! 👍

Copy link
Contributor

@kevinsawicki kevinsawicki left a comment

Choose a reason for hiding this comment

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

LGTM

@zeke
Copy link
Member Author

zeke commented Aug 8, 2017

Update: Closed all open issues, encouraging folks to open followup issues on electron/electron if needed.

@malept
Copy link
Member

malept commented Aug 9, 2017

It'd be nice if there was a bot that auto-closed new issues/PRs (with roughly the same message as @zeke's), but not required. Mostly because people have a tendency to automatically delete PR/issue template text, from my experience.

@zeke
Copy link
Member Author

zeke commented Aug 9, 2017

people have a tendency to automatically delete PR/issue template text

Let's wait it out and see.

It'd be nice if there was a bot that auto-closed new issues/PRs

WIP here: https://github.com/electron/electron-probot

@malept
Copy link
Member

malept commented Aug 9, 2017

WIP here: https://github.com/electron/electron-probot

I think it's currently private.

@zeke
Copy link
Member Author

zeke commented Aug 9, 2017

🔒 🔨

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants