-
Notifications
You must be signed in to change notification settings - Fork 439
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
Reactive python 3.6 in gitsome #135
Conversation
Codecov Report
@@ Coverage Diff @@
## master #135 +/- ##
=======================================
Coverage 79.23% 79.23%
=======================================
Files 36 36
Lines 2817 2817
=======================================
Hits 2232 2232
Misses 585 585 |
Hi @szlin, I recall there were issues running in Python 3.6 because gitsome has a bundled version of xonsh, instead of listing it as a dependency. I'm not seeing a change in this PR that addresses that? |
@donnemartin Do you mind to remove embedded code copies and add dependency with xonsh? I am the gitsome package maintainer in Debian, and the copies of code need to be removed in Debian archive according to Debian policy 4.13, hence I had created a patch for removing xonsh - Removal xonsh patch in Debian |
This is something I'm hoping to get fixed at some point with #25, I recall at the time there were issues trying to unbundle xonsh. It sounds like you've already done this, did you face any issues? |
@donnemartin I haven't seen any issues and received any bug reports so far since 2017-02-20. You may reference to following link: gitsome in Debian |
+1 It would be nice to unbundle Regards. |
Looking for that getting fixed/merged for months! Great job guys :) Congrats. Waiting happily ~ |
Any news? |
Hi everyone, I'm seeing issues with this, see #105. |
@donnemartin The gitsome with my patches works fine for me, and got no bug report both in Debian [1] and Ubuntu [2] over past 10 months. Could anyone provide the reproducible steps for python3.6 error in gitsome after applying my patches? I can have a look afterwards.
[1] https://tracker.debian.org/pkg/gitsome |
1.5 years since last release and a lot of people want 3.6 support. Contributors have sent a pull request. I request @donnemartin to please have a look, approve and merge it. If you see a problem with it, please convey it so that one the OP or someone else may address it. I understand that you might be busy with other things in your life, hence, you may think of adding some other maintainers on the project so it takes the load off your head, maybe? People love this awesome tool but it is not moving forward |
Hey all, sorry been really busy. I'm starting to free up and plan to look into updating gitsome to support the latest Python versions. |
Fixed with #160, gitsome now support Python 3.7. |
This PR is trying to fix #105 and reactive python 3.6 in gitsome.
Since xonsh has fixed the issue in python 3.6, the gitsome should work fine with python 3.6. I had some tests done on my side, please review on it.
Thanks.