Add function “reference()” to access a PhysicalConstant’s reference

[Socob]

Not exported due to the potentially generic name, but I don’t think it necessarily needs to be.

Fixes #35.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.19%. Comparing base (72782d8) to head (6d1607e).
Report is 1 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff             @@
##           master      #37       +/-   ##
===========================================
- Coverage   93.75%   77.19%   -16.56%     
===========================================
  Files           3        3               
  Lines          80       57       -23     
===========================================
- Hits           75       44       -31     
- Misses          5       13        +8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[giordano]

Can you please add a test for this? Agreed not to export the generic name

[Socob]

Ah, I thought the doctest was enough. Maybe the doctests should be run as part of regular testing?

[giordano]

I prefer to have explicit tests besides doctests 🙂

[Socob]

It wouldn’t hurt having them run automatically in addition to other tests, though?

Either way, tests added!

[Socob]

No idea why Codecov thinks that codata2018.jl was covered 100% before and is now 0%…

[giordano]

Doctests are run as part of the documentation building, so they're still run and would fail the build in case something break anyways. But I like keeping tests self-sufficient, too.