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

support ConstructionBase #41

Merged
merged 1 commit into from
May 17, 2022
Merged

support ConstructionBase #41

merged 1 commit into from
May 17, 2022

Conversation

aplavin
Copy link
Contributor

@aplavin aplavin commented May 14, 2022

manual implementation only needed for FK5Coords{e} due to the type parameter, other types work out of the box
allows interop with packages listed at https://github.com/JuliaObjects/ConstructionBase.jl

manual implementation only needed for FK5Coords{e}, other types work out of the box
allows interop with packages listed at https://github.com/JuliaObjects/ConstructionBase.jl
@codecov
Copy link

codecov bot commented May 14, 2022

Codecov Report

❗ No coverage uploaded for pull request base (master@10cc6dc). Click here to learn what that means.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master      #41   +/-   ##
=========================================
  Coverage          ?   97.91%           
=========================================
  Files             ?        2           
  Lines             ?       96           
  Branches          ?        0           
=========================================
  Hits              ?       94           
  Misses            ?        2           
  Partials          ?        0           
Impacted Files Coverage Δ
src/SkyCoords.jl 97.29% <ø> (ø)
src/types.jl 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 10cc6dc...a4d528d. Read the comment docs.

Copy link
Member

@mileslucas mileslucas left a comment

Choose a reason for hiding this comment

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

Looks good to me

@giordano giordano merged commit 704857f into JuliaAstro:master May 17, 2022
@aplavin
Copy link
Contributor Author

aplavin commented Jun 24, 2022

@giordano @mileslucas would you please make a new release with this change?

@mileslucas
Copy link
Member

@aplavin JuliaRegistries/General#62994

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.

3 participants