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

Incorporating @6StringJazzer's edits into master #40

Merged
merged 6 commits into from
Jun 7, 2018
Merged

Conversation

Koosmann
Copy link
Collaborator

@Koosmann Koosmann commented Jun 7, 2018

No description provided.

6StringJazzer and others added 6 commits March 26, 2018 11:16
Removed "!" after Pi and e constant names. There is no need to show excitement there, and even worse it looks like the factorial operator. Also e should never be uppercase when talking about the mathematical constant.
They show examples for pi. Code change proposes appropriate examples.
1. State that this example is to be used in the Transparency field
2. Transparency value for total opaqueness is 100, not 1.
Examples for ceil and floor functions are incorrect.
@Koosmann Koosmann changed the title Incorporating @6stringjazzer's edits into master Incorporating @6StringJazzer's edits into master Jun 7, 2018
@Koosmann
Copy link
Collaborator Author

Koosmann commented Jun 7, 2018

Thanks @6StringJazzer !

@Koosmann Koosmann merged commit 3751cdc into master Jun 7, 2018
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.

2 participants