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

Make the power tests pass #36

Merged
merged 10 commits into from Nov 11, 2012
Merged

Make the power tests pass #36

merged 10 commits into from Nov 11, 2012

Conversation

ghost
Copy link

@ghost ghost commented Nov 11, 2012

There are a few todos around the return values not matching the docs but the tests need them to pass so will need to find the proper set of flags from somewhere. Also made it not use the param stuff so much, which may or may not need closer looking at in the future.

xsacha and others added 10 commits November 11, 2012 21:32
Improves rendering of skinning demo.
The end of the tube is wrong and collapses on the origin.
Removed PARAM stuff, made tests pass with some caveats in that I don't
understand what the return values are supposed to be based on the docs
and fix a small issue with sceImpose that I created with the last merge
@hrydgard
Copy link
Owner

Oops, I fixed this too in the meantime :) I'll have to do some conflict resolving, i like your param improvements.

@ghost
Copy link
Author

ghost commented Nov 11, 2012

haha, typical :)

Let me know if I can help.

@hrydgard hrydgard merged commit a2c024e into hrydgard:master Nov 11, 2012
@hrydgard
Copy link
Owner

Done

@hrydgard
Copy link
Owner

Btw, rebase on master the next time you send a pull request, we ended up with some duplicates commits in the history now :)

@ghost
Copy link
Author

ghost commented Nov 11, 2012

Yes. I'm quite bad at git so thanks for pointing that out. Probably should have read the docs before sending a pull request ;)

@ghost
Copy link
Author

ghost commented Nov 12, 2012

Is there an easy way to rebase to your master branch? Or is it just a case of deleting my fork? If I try and do:
git update upstream
git merge upstream/master

it ends up duplicating the commits. I've been using source control forever and this is making me feel very stupid :)

@Jessidhia
Copy link
Contributor

Git has a concept of an "upstream" branch, and whenever you run most updating commands without arguments, it does the right thing. Set your branch's upstream using git branch --set-upstream master upstream/master.

Since you were doing development on master, when updating the code you should do git pull --rebase, which automatically deduplicates commits, and lets you fix merge conflicts commit-by-commit. Ideally, you should do development on a so-called "topic" branch. See http://nvie.com/posts/a-successful-git-branching-model/ for an example of git workflow.

@ghost
Copy link
Author

ghost commented Nov 12, 2012

That actually makes a lot of sense, especially the topic branch idea. I'll give it a go next time. Thanks!

@hrydgard
Copy link
Owner

I need to start using topic branches more, too :) Thanks for the explanation Kovensky.

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.

4 participants