-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Release v1.2.1 #1906
Conversation
@jviotti mind taking that high-res screenshot again? :) |
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 |
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.
Huh? According to https://github.com/resin-io/etcher/commits/master this was a feature in v1.2.0 ??
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.
That's because the release branch itself was only merged after; but that definitely didn't end up in v1.2.0
1abe8f7
to
b15183a
Compare
@jhermsmeier Looks like you should get yourself a company brand new Retina MacBook Pro, for "screenshooting" purposes :P |
...or you could just write a little ImageMagick script to programatically redraw the version number in the bottom-right corner of the current screenshot ;-) |
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 |
@lurch why would we need to publish the AppImages to S3 as well? |
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) |
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. |
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.
Can we remove the trailing period here?
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.
Done
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.
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.
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.
Left one small comments regarding the CHANGELOG, but looks good otherwise
b15183a
to
7c9aa6d
Compare
I wonder if Spectron supports emulating high-res Retina displays? |
PR Checklist:
package.json
'sversion
propertynpm-shrinkwrap.json
'sversion
propertyCHANGELOG.md
by runningmake changelog
screenshot.png
so it displays the latest version in the bottomright corner
updates.semverRange
version inpackage.json
Release Checklist: