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.

Sign up for GitHub

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

Jump to bottom

Fix typo #1562

Merged

yorikvanhavre merged 1 commit into FreeCAD:master from mwganson:master

Jul 23, 2018

Contributor

mwganson commented Jul 20, 2018

I figure this should be Z instead of Y, but even if I'm wrong there's no reason to check for hasattr(pts,'Y') twice.

Thank you for creating a pull request to contribute to FreeCAD! To ease integration, please confirm the following:

Branch rebased on latest master git pull --rebase upstream master
Unit tests confirmed to pass by running ./bin/FreeCAD --run-test 0
Commit message is well-written
Commit message includes issue #<id> or fixes #<id> where <id> is the associated MantisBT issue id if one exists

And please remember to update the Wiki with the features added or changed once this PR is merged.


          Fix typo

de97277

I figure this should be Z instead of Y, but even if I'm wrong there's no reason to check for hasattr(pts,'Y') twice.

yorikvanhavre merged commit 5a39efa into FreeCAD:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment