Skip to content

Re-export atan2 from GHC.Float#165

Merged
chshersh merged 1 commit intokowainik:masterfrom
ethercrow:patch-1
Jun 17, 2019
Merged

Re-export atan2 from GHC.Float#165
chshersh merged 1 commit intokowainik:masterfrom
ethercrow:patch-1

Conversation

@ethercrow
Copy link
Copy Markdown
Contributor

@ethercrow ethercrow commented Jun 15, 2019

Doing geometry without atan2 makes Jack a dull boy.

Checklist:

HLint

  • I've changed the exposed interface (add new reexports, remove reexports, rename reexported things, etc.).
    • I've updated hlint.dhall accordingly to my changes (add new rules for the new imports, remove old ones, when they are outdated, etc.).
    • I've generated the new .hlint.yaml file (see this instructions).

General

  • I've updated the CHANGELOG with the short description of my latest changes.
  • All new and existing tests pass.
  • I keep the code style used in the files I've changed (see style-guide for more details).
  • I've used the stylish-haskell file.
  • My change requires the documentation updates.
    • I've updated the documentation accordingly.
  • I've added the [ci skip] text to the docs-only related commit's name.

@vrom911 vrom911 added the reexport Reexport something new label Jun 16, 2019
import GHC.Base (maxInt, minInt)
import GHC.Float (Double (..), Float (..), Floating (acos, acosh, asin, asinh, atan, atanh, cos, cosh, exp, logBase, pi, sin, sinh, sqrt, tan, tanh, (**)),
RealFloat (floatRadix, floatDigits, floatRange, decodeFloat, encodeFloat, isNaN, isInfinite, isDenormalized, isNegativeZero, isIEEE))
RealFloat (floatRadix, floatDigits, floatRange, decodeFloat, encodeFloat, isNaN, isInfinite, isDenormalized, isNegativeZero, isIEEE), atan2)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see that atan2 is the method of the class RealFloat, you can add the import under the class as all the other functions here.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Copy link
Copy Markdown
Contributor

@chshersh chshersh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@chshersh chshersh merged commit 559ed98 into kowainik:master Jun 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

reexport Reexport something new

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants