Release version 3.8 (#207)

whimboo commented 8 years ago

This PR is the fix for issue #207 and adds the history changes and bumps the version number.

@xabolcs can you please review? Thanks

UtiluMark commented 8 years ago

Shouldn't Seamonkey's maxVersion be 2.48 instead of 2.40?

UtiluMark commented 8 years ago

Notes on the changelog:

In previous versions each line was indented with two spaces " ", but the 3.8 changes are indented with only one space " ".
In previous versions each line was prefixed with either "Repository: " or "Extension: ", but the 3.8 changes don't have prefixes.
"Release version 3.8 (#207)" isn't worth mentioning in the history of changes I think, none earlier version has a similar release line.
"Update minVersions for applications (#207)" would be worth mentioning.

tonymec commented 8 years ago

FWIW:

2.40 is the current stable version of SeaMonkey, which is getting a little old in the teeth.
The developers hope to be able to release SeaMonkey 2.45 "soon" but there is no firm date yet. Pre-release builds are already available in subdirectories of http://ftp.mozilla.org/pub/seamonkey/tinderbox-builds/ with subdirectory names starting "comm-release-" followed by "linux" "linux64" "macosx64" or "trunk-win32" depending on platform. After selecting the appropriate page for your platform, scroll to the bottom of the page, then select the numbered subfolder with the highest number. -- SeaMonkey 2.45 will include Lightning and Devtools; its pre-releases already do.
2.46 is the current beta version.
2.47a2 is the current Aurora version.
2.48a1 is the current Trunk version.

whimboo commented 8 years ago

In previous versions each line was indented with two spaces " ", but the 3.8 changes are indented with only one space " ".

We should better correct the older entries as continuing with two spaces.

In previous versions each line was prefixed with either "Repository: " or "Extension: ", but the 3.8 changes don't have prefixes.

I feel this is obsolete information given that most changes will affect the extension. We could save us from this hand-crufting additions for each version by specifying explicitly if a change was done to the repository, and remove all 'Extension' prefixes. Does that sound ok?

"Release version 3.8 (#207)" isn't worth mentioning in the history of changes I think, none earlier version has a similar release line. "Update minVersions for applications (#207)" would be worth mentioning.

Ok, I will update this line so we mention the min version bump.

Also thanks for pointing out the current to hopefully be released version of SeaMonkey.

I will update the PR once we reached consensus about the prefixes.

xabolcs commented 8 years ago

We should better correct the older entries as continuing with two spaces.

I'm OK with changing the old entries.

I feel this is obsolete information given that most changes will affect the extension. We could save us from this hand-crufting additions for each version by specifying explicitly if a change was done to the repository, and remove all 'Extension' prefixes. Does that sound ok?

It's OK to me to remove all the Extension: prefixes.

whimboo commented 8 years ago

I pushed an update for this PR. So please check again. Thanks.

xabolcs commented 8 years ago

Thanks! Looks good, only one note about .editorconfig. See inline comment!

whimboo commented 8 years ago

An update has been pushed. Will land the patch now.

mozilla / nightlytt

Release version 3.8 (#207) #208