These are chat archives for EntilZha/PyFunctional

8th
May 2015
Adrian Wielgosik
@adrian17
May 08 2015 16:10
Um, why did you rebase that branch instead of merging? This only worked nicely because there were no conflicts, but otherwise it could be a bit of a pain for people pulling from master
Pedro Rodriguez
@EntilZha
May 08 2015 17:28
Not sure why it would be a pain from pulling from master. To anyone working on master, it should appear like additional commits were made on master
Mostly, for a merge there would be a large diff I think, and probably non-trivial merge conflicts. Since I know the changes in master are evolutionary (not wiping out big pieces of code), I can pick up most of those changes and treat it like I was just writing lineage on master (didn't originally because it was experimental). Fixing conflicts turned out to be pretty easy and only in one place (tests that Digenis added)
Did it cause problems for you?
Adrian Wielgosik
@adrian17
May 08 2015 17:37
no, it did not
One smaller issue is that these additional commits are different from the ones made on the branch
Pedro Rodriguez
@EntilZha
May 08 2015 17:37
yay, chatting
Pedro Rodriguez
@EntilZha
May 08 2015 17:43
you think changes on master are ready for a release?
Adrian Wielgosik
@adrian17
May 08 2015 17:47
(And if someone had a commit on top of the other branch, he couldn't easily merge/rebase it into master)
I didn't test it too much
But since everything seems to be working fine...
Hm, what about dependency on enum package for Python2?
Pedro Rodriguez
@EntilZha
May 08 2015 17:51
There may be a way to not use that, but it is convinient
I suppose, I thought more of hte lineage-rewrite branch as a workspace for me, to keep master always passing
so virtually working on master sortof
Adrian Wielgosik
@adrian17
May 08 2015 17:51
Ah, it's in the setup file... I didn't realize it has this list separately from requirements.txt
Pedro Rodriguez
@EntilZha
May 08 2015 17:52
Ya, I don't think there is a requirements file
Could probably put that stuff outside setup.py and into a file like that
Adrian Wielgosik
@adrian17
May 08 2015 17:54
Nah, there's no need to
Pedro Rodriguez
@EntilZha
May 08 2015 17:55
I would like to redo the homepage, I have some ideas to improve that
Adrian Wielgosik
@adrian17
May 08 2015 17:56
Oh, did readthedocs break?
Pedro Rodriguez
@EntilZha
May 08 2015 17:56
probably something isn't right
need to take a look at that
curious, how much do you use the package?
docs seem fine
One personal project, a couple of one-time scripts, not much really
Pedro Rodriguez
@EntilZha
May 08 2015 17:59
hmm, will take a look at it
see whats going on
i haven't been using it too much recently, that being said haven't been doing much that would need it (web django and docker stuff)
Adrian Wielgosik
@adrian17
May 08 2015 18:00
Oh, one thing I wanted to ask about, maybe make a wrapper for range so instead of seq(range(100)).stuff it would be possible to write seq.range(100).stuff?
Pedro Rodriguez
@EntilZha
May 08 2015 18:00
would be pretty easy to do
I just added this morning, but not pushed yet, making it so you can do seq(1, 2, 3, 4).map(etetc)
Pedro Rodriguez
@EntilZha
May 08 2015 19:38
docs should be fixed now, it was missing a requirement.