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

Tqdm2 #35

Merged
merged 61 commits into from
Oct 11, 2015
Merged

Tqdm2 #35

merged 61 commits into from
Oct 11, 2015

Conversation

casperdcl
Copy link
Sponsor Member

local branch migrated from #34

Signed-off-by: Stephen L. <lrq3000@gmail.com>
Signed-off-by: Stephen L. <lrq3000@gmail.com>
Signed-off-by: Stephen L. <lrq3000@gmail.com>
Signed-off-by: Stephen L. <lrq3000@gmail.com>
…g 100% branch coverage!)

Signed-off-by: Stephen L. <lrq3000@gmail.com>
Signed-off-by: Stephen L. <lrq3000@gmail.com>
Signed-off-by: Stephen L. <lrq3000@gmail.com>
Signed-off-by: Stephen L. <lrq3000@gmail.com>
Signed-off-by: Stephen L. <lrq3000@gmail.com>
@hadim
Copy link
Contributor

hadim commented Oct 11, 2015

@lrq3000 are you doing the modif and do you want me to take care of it ?

@casperdcl
Copy link
Sponsor Member Author

pandoc basically rules my life, I was going to use it to convert everything to rst and then manually tweak anything that was broken. Personally I prefer md, though... but since pypi doesn't support it we don't really have a choice.

@lrq3000
Copy link
Member

lrq3000 commented Oct 11, 2015

You can do the stuff about version, I can do the rest :)

@hadim
Copy link
Contributor

hadim commented Oct 11, 2015

Well I am currently writing my PhD thesis with pandoc @casperdcl :-)

What do we do with version finally ?

Signed-off-by: Stephen L. <lrq3000@gmail.com>
@lrq3000
Copy link
Member

lrq3000 commented Oct 11, 2015

@kmike is against reading from a file, so I guess we will use the AST.

@lrq3000
Copy link
Member

lrq3000 commented Oct 11, 2015

Ehm... guys, could someone tell me how I can contribute to this branch? I cannot use my credentials...

@hadim
Copy link
Contributor

hadim commented Oct 11, 2015

git co tqdm2 then you can commit and push to it

@lrq3000
Copy link
Member

lrq3000 commented Oct 11, 2015

Yes I checked out but I cannot push directly on tqdm/tqdm, my usual credentials are refused.

@hadim
Copy link
Contributor

hadim commented Oct 11, 2015

that should work now (I modified your right in the org settings)

@lrq3000
Copy link
Member

lrq3000 commented Oct 11, 2015

Ok all good, I had to use my private key password, which is a bit weird but ok.

@lrq3000
Copy link
Member

lrq3000 commented Oct 11, 2015

Ah ok great, that's why it worked :) Thank's!

@lrq3000
Copy link
Member

lrq3000 commented Oct 11, 2015

I'm working on converting the Markdown Readme to ReStructured Text

@hadim
Copy link
Contributor

hadim commented Oct 11, 2015

I guess we can live with that but be carefull next time and overall never force push on master !!!

@lrq3000
Copy link
Member

lrq3000 commented Oct 11, 2015

@casperdcl rebased it to clean history. You can try to git pull --rebase to fix your issue, or else delete and reclone the whole project :/ Anyway, the 2.0.0 tag is still referencing the correct commit, so it should be OK.

@lrq3000
Copy link
Member

lrq3000 commented Oct 11, 2015

Arg no you're right, it messed up the repo:

"lrq3000 released this an hour ago · 52 commits to master since this release"

Not cool... @casperdcl, do you think this is fixable?

@casperdcl
Copy link
Sponsor Member Author

yeah didn't realise you'd released already.

@hadim
Copy link
Contributor

hadim commented Oct 11, 2015

Well unless I am missing something git tag is for 4d7aac9 and this commit never shows in the master branch...

@lrq3000
Copy link
Member

lrq3000 commented Oct 11, 2015

Yes @hadim, you're right, it points to a non existent reference now. We can still guess what was the history state because we still have the correct commit message, but the commit hash is pretty much useless now...

@casperdcl
Copy link
Sponsor Member Author

would clearing all tags and then tagging the last commit v2.0.0 work?

@hadim
Copy link
Contributor

hadim commented Oct 11, 2015

@casperdcl seems to be the only solution... please never force push in master again :-D

@hadim
Copy link
Contributor

hadim commented Oct 11, 2015

I let you do it ( @lrq3000 or @casperdcl )

@lrq3000
Copy link
Member

lrq3000 commented Oct 11, 2015

Yes, I'm doing that since I have the builds

@lrq3000
Copy link
Member

lrq3000 commented Oct 11, 2015

AH! well I have another solution: I can force push the entire history back since I still have it on my computer. But I don't know what is the most messy option between the two lol.

@hadim
Copy link
Contributor

hadim commented Oct 11, 2015

If you still have the correct history, please force push it !

@casperdcl
Copy link
Sponsor Member Author

Was really meant to be rebase (bviously incl force-push) on the tqdm2 branch. really was surprised that you merged it in.

@hadim
Copy link
Contributor

hadim commented Oct 11, 2015

Be sure the v2 tag hash is in your history

@hadim
Copy link
Contributor

hadim commented Oct 11, 2015

@casperdcl yeah its not only your fault here, I also totally forgot that you were rebasing... sorry for that

@hadim
Copy link
Contributor

hadim commented Oct 11, 2015

Next time we should first merge the PR, create the git tag and then upload to pypi (in that order).

@casperdcl
Copy link
Sponsor Member Author

nvm. i don't mind re-rebasing if you still have old history. obviously this time i'll leave in the appropriate commit... or we could just leave the messy history as is? idk.

I would think just re-tagging is best since it keeps the clean history?

@hadim
Copy link
Contributor

hadim commented Oct 11, 2015

Well I don't what's best, I let you decide with @lrq3000

@lrq3000
Copy link
Member

lrq3000 commented Oct 11, 2015

Yes the old hash is in my history so normally I can recover the whole old history.

Seriously guys I don't know lol, I just have a bad feeling about the new history being rebased after releasing stuff and all, but I don't know, we can technically keep it.

@hadim
Copy link
Contributor

hadim commented Oct 11, 2015

Well seems weird to me too so let just not rebasing this time and keep the old and correct history no ?

@lrq3000
Copy link
Member

lrq3000 commented Oct 11, 2015

I follow my intuition, I push back the old history and @casperdcl you push your new changes on top. We will be careful next time to rebase and do all git maintenance before releasing.

@hadim
Copy link
Contributor

hadim commented Oct 11, 2015

he cant rebase without re writing the history...

@lrq3000
Copy link
Member

lrq3000 commented Oct 11, 2015

OK, old history restored. The release tag seems to be restored OK

@lrq3000
Copy link
Member

lrq3000 commented Oct 11, 2015

Yes but he did some additional changes to RELEASE and some other stuff, that's what I meant

@hadim
Copy link
Contributor

hadim commented Oct 11, 2015

Ok it will go in the next release i guess

@lrq3000
Copy link
Member

lrq3000 commented Oct 11, 2015

Yes anyway we can always make things better :) So we would never release if we waited the project to be perfect ;)

@kmike
Copy link
Contributor

kmike commented Oct 11, 2015

We will be careful next time to rebase and do all git maintenance before releasing

Even better to do rebasing only before merging PRs. It seems this time we just had a really bad luck (quick rebase of merged PR), it wasn't anyone's error.

@casperdcl
Copy link
Sponsor Member Author

I'd rather say it was everyone's fault (incl. mine). I think we were all too eager having finally managed to contribute to the project proper. anyway nice work.

@lrq3000
Copy link
Member

lrq3000 commented Oct 11, 2015

Yes we'll do better next time :) The release is good enough, it will be better next time, but I'm already quite glad the project reached this stage of maturity.

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

4 participants