-
Notifications
You must be signed in to change notification settings - Fork 209
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
Cut a new bugwarrior release (2.0.0) #339
Comments
Any objections? Should we wait for any pending fixes first? |
I'll be ready to upload a package to Debian as soon as the release is cut. |
@ralphbean Given that we don't do point releases, cutting a release right after a bunch of changes and before the development branch has been tested out by multiple developers doesn't make much sense to me. Seems pretty likely that we've introduced some silly regressions. My thinking is that if you don't do point releases, all releases should be stable. Therefore it makes sense to cut a release when the development branch has been quiet for a while. I don't really care though, because I always use a development version. Your call. |
#330 is a problem. I can produce simple man pages though, as these are not complex commands. The bug is resolved in Debian by having a man page, not by having complete documentation in the man page. The man pages just need to describe basically what the command does and where to find the full docs (which are installed in HTML format by the Debian package also). |
Ok, wait a minute. I broke a thing in the Trac tests (test is broken, not the code). #341 or similar should definitely be merged before a new release. |
Also, any idea what broke Travis? |
Seems to be some sort of missing dependency. |
I'm fine with waiting a week.
I'd rather not add a man page to this repo if we can (I like help2man for things like this). But if that doesn't work for some reason, of course we could accept a manpage here. |
help2man is fine. (: |
Checking in. Any concerns with cutting a release sometime this week? |
I finally got around to testing the latest changes. Assuming it's a valid bug, it would be nice to fix #345 first. |
Sounds like #350 is not a regression so I'd say go for it. |
@ralphbean Per @gdetrez's suggestion, if you cut a release in the near future I'd recommend checking out 1aa6828 (the last commit before python3 compatibility) since there are likely still some python2 regressions we haven't yet discovered and we certainly haven't fixed all the python3 bugs yet. |
@ralphbean If we're going to release with Python 3 support, I'd be changing the Debian package to Python 3 and there may be enough changes here that maybe we call it 2.0.0? Not sure about the roadmap, just a suggestion. |
2.0.0 sounds like the right target at this point. Let's keep it in develop for a little bit longer since we just mixed things up with Python3. :) |
@ralphbean Just checking in on the progress, and a reminder that Debian will be coming up to a freeze and if 2.0.0 is to make it into Debian stretch, it needs to be soon. |
Hey team, any feedback on the python3 stability? Are we looking safe for a 2.0.0 release? |
I doubt very many people have tested out python3 yet. However, I'm not sure that's a problem as long as python2 is stable (which I believe it is). I'd recommend packagers to continue to use python2, but the reality is most people are going to pip install with whatever their distro ships by default -- the best we can do for them is give them the best version of bugwarrior we have, which I believe is in |
Also, it would be nice if #395 made it in before release. |
ETA? :) Looking forward to the py3 support! For what I can see, the issues discussed above are more or less solved, except the man pages being a little unstructured. https://github.com/ralphbean/bugwarrior/network - |
@xeor Yes, |
OK, I ended up cutting 1.5.1, available here: https://pypi.python.org/pypi/bugwarrior/1.5.1 |
I think we're about due.
The text was updated successfully, but these errors were encountered: