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

Add release notes for 1.4.1 #656

Merged
merged 3 commits into from
Apr 1, 2017
Merged

Conversation

sambhav
Copy link
Collaborator

@sambhav sambhav commented Mar 13, 2017

No description provided.

@sambhav sambhav force-pushed the 1.4.1releasenotes branch from 2a925a2 to 322ca89 Compare March 13, 2017 13:32
@sambhav sambhav force-pushed the 1.4.1releasenotes branch from 322ca89 to b4b9c65 Compare March 13, 2017 13:33
@Sophist-UK
Copy link
Contributor

#604 was promised to be included in 1.4.1. Is this still going to be included?

@sambhav
Copy link
Collaborator Author

sambhav commented Mar 13, 2017

The issue mentioned by @mineo in #613 (comment) still exists afaict, 1.4.1 is now on a feature freeze anyway and we'd be migrating to qt5 soon so that PR will need a rewrite.

@Sophist-UK
Copy link
Contributor

Sorry if there is some confusion. I mentioned #604 not #613.

@sambhav
Copy link
Collaborator Author

sambhav commented Mar 13, 2017

#604 (comment)

@Sophist-UK
Copy link
Contributor

Point 1 of the #613 (comment) doesn't apply because it relates to features of #613 which are not in #604.

I will take a look at Point 2 again, though I answered @mineo's comments about it perhaps being related to o/s and several comments after that asking that the PR be merged so it could have more extensive testing on non-Windows.

#604 gives a lot of benefits and now it won;t make 1.4.1.

More to the point it is a significant PR which had a lot of my effort and, since I don't have the time to rewrite it for QT5, if there is not gong to be a 1.4.2 it can be included in before you move to v2 with QT5 then this much needed fix is not going to happen,

Might I suggest that it might be better to create a v2 branch and continue development in parallel at least until we get to a beta version of v2. This seems to me to be the point that we can functionally stabilise 1.4, retrofit any 1.4 fixes to v2 and ask people to start coding new PRs against v2 rather than v1.4.

@Sophist-UK
Copy link
Contributor

Or perhaps we can merge it after 1.4.1 in order to take it into whatever the next release is - 1.4.2 or v2

@sambhav
Copy link
Collaborator Author

sambhav commented Mar 13, 2017

@Sophist-UK the plan is that we will make 2.0dev the master and continue the dev for 1.4.X in another branch. You can definitely have it in 1.4.X, in case you don't have the time to port it to Qt5. Albeit after it has been tested and approved by other devs to be working on all 3 major OSes :)

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@Sophist-UK
Copy link
Contributor

I will look again at the problem reported by @Mimeo (even though I answered his issue at the time) but if it is specific to an o/s I don;t have then I can't fix it.

My point is that sometimes you need to merge it knowing it is imperfect but better than current in order to then improve it still further.

As for branches, IMO it would still be better for this to be merged before 2.0dev is split off even though it is imperfect because the imperfections can then be ironed out.

@zas zas merged commit 1e94ac7 into metabrainz:master Apr 1, 2017
@sambhav sambhav deleted the 1.4.1releasenotes branch February 9, 2018 22:04
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.

None yet

3 participants