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 v1.2.1 #1906

Merged
merged 1 commit into from
Dec 7, 2017
Merged

Release v1.2.1 #1906

merged 1 commit into from
Dec 7, 2017

Conversation

jhermsmeier
Copy link
Contributor

@jhermsmeier jhermsmeier commented Dec 6, 2017

PR Checklist:

  • Bump the version number in the package.json's version property
  • Bump the version number in the npm-shrinkwrap.json's version property
  • Add a new entry to CHANGELOG.md by running make changelog
  • Re-take screenshot.png so it displays the latest version in the bottom
    right corner
  • Revise the updates.semverRange version in package.json

Release Checklist:

  • Manual regression testing
  • Merge this
  • Tag the commit on master
  • Publish the draft release
  • Publish packages on Bintray
  • Publish packages on S3
  • Forum post
  • Update Etcher website
  • Mailing List update

@jhermsmeier
Copy link
Contributor Author

@jviotti mind taking that high-res screenshot again? :)

@jhermsmeier jhermsmeier added this to the v1.2.1 milestone Dec 6, 2017
CHANGELOG.md Outdated

- Fix handling of temporary read/write errors
- Don't send initial Mixpanel events before "Anonymous Tracking" settings are loaded.
- Fix verification step reading from the cache
Copy link
Contributor

Choose a reason for hiding this comment

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

Huh? According to https://github.com/resin-io/etcher/commits/master this was a feature in v1.2.0 ??

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's because the release branch itself was only merged after; but that definitely didn't end up in v1.2.0

@jviotti
Copy link
Contributor

jviotti commented Dec 6, 2017

screen shot 2017-12-06 at 1 52 31 pm

@jviotti
Copy link
Contributor

jviotti commented Dec 6, 2017

@jhermsmeier Looks like you should get yourself a company brand new Retina MacBook Pro, for "screenshooting" purposes :P

@lurch
Copy link
Contributor

lurch commented Dec 7, 2017

...or you could just write a little ImageMagick script to programatically redraw the version number in the bottom-right corner of the current screenshot ;-)

@lurch
Copy link
Contributor

lurch commented Dec 7, 2017

Publish the draft release
Publish packages on S3

If we want to 'fix' #1893 with Etcher v1.2.1 (I don't think there's any reason not to?), then you just need to make this change locally

--- a/Makefile
+++ b/Makefile
@@ -423,6 +423,7 @@ TARGETS +=  \
        electron-installer-redhat \
        cli-installer-tar-gz
 PUBLISH_AWS_S3 += \
+       $(BUILD_DIRECTORY)/$(APPLICATION_NAME_LOWERCASE)-$(APPLICATION_VERSION)-$(TARGET_ARCH_APPIMAGE).AppImage \
        $(BUILD_DIRECTORY)/$(APPLICATION_NAME_LOWERCASE)-$(APPLICATION_VERSION)-$(PLATFORM)-$(TARGET_ARCH_APPIMAGE).zip \
        $(BUILD_DIRECTORY)/$(APPLICATION_NAME)-cli-$(APPLICATION_VERSION)-$(PLATFORM)-$(TARGET_ARCH).tar.gz
 PUBLISH_BINTRAY_DEBIAN += \

before running make publish-aws-s3 (and I'll make the corresponding change to #1896 ). And then remember to also add the unzipped AppImages to the draft GitHub release.
EDIT: Aha, I see you've already done the last part :)

@jhermsmeier
Copy link
Contributor Author

@lurch why would we need to publish the AppImages to S3 as well?

@lurch
Copy link
Contributor

lurch commented Dec 7, 2017

I don't think there's any real reason, it just feels more consistent. And if we add the change to the Makefile, then future snapshot uploads of Etcher will also have the unzipped AppImages available (since the snapshot releases currently only go to S3)

@jviotti
Copy link
Contributor

jviotti commented Dec 7, 2017

...or you could just write a little ImageMagick script to programatically redraw the version number in the bottom-right corner of the current screenshot ;-)

We could do this with a small Spectron script btw

CHANGELOG.md Outdated
### Fixes

- Fix handling of temporary read/write errors
- Don't send initial Mixpanel events before "Anonymous Tracking" settings are loaded.
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we remove the trailing period here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh, I assumed that the CHANGELOG.md was only auto-generated by versionist, and there was no scope for hand-editing it? If we are "allowed" to hand-edit it, then I'd suggest removing "Add a sudo-prompt upon launch on Linux-based systems." as that change actually got reverted when we removed the Linux usbboot support, and it never made it into a published version of Etcher.

Copy link
Contributor

@jviotti jviotti left a comment

Choose a reason for hiding this comment

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

Left one small comments regarding the CHANGELOG, but looks good otherwise

@lurch
Copy link
Contributor

lurch commented Dec 7, 2017

We could do this with a small Spectron script btw

I wonder if Spectron supports emulating high-res Retina displays?

@jhermsmeier jhermsmeier merged commit 4c00a25 into master Dec 7, 2017
@jhermsmeier jhermsmeier deleted the release-1.2.1 branch December 7, 2017 15:44
jviotti pushed a commit that referenced this pull request Dec 14, 2017
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.

3 participants