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

Spelling mistakes in Class Summary documentation #658

Closed
danieljuschus opened this issue Feb 24, 2021 · 1 comment · Fixed by #659
Closed

Spelling mistakes in Class Summary documentation #658

danieljuschus opened this issue Feb 24, 2021 · 1 comment · Fixed by #659

Comments

@danieljuschus
Copy link
Contributor

The word "wither" appears three times on https://cadquery.readthedocs.io/en/latest/classreference.html. I believe this should be "whether", right? I'm quite new to open source software, can I submit a pull request myself by forking and editing this in https://github.com/CadQuery/cadquery/blob/master/cadquery/cq.py?

@jmwright
Copy link
Member

I believe this should be "whether", right?

Yes, good catch.

I'm quite new to open source software

Welcome.

can I submit a pull request myself by forking and editing this in

Yes, you fork the cadquery repo, clone your fork to your computer, make the changes, push those changes to your fork, then open a PR. We'll work with you on the PR if anything needs to be changed (our core dev team is welcoming of newcomers and PRs), then we'll merge. I can answer other questions about the process if you have any.

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 a pull request may close this issue.

2 participants