Skip to content
This repository has been archived by the owner on Aug 12, 2022. It is now read-only.

Register CoupledFields [https://github.com/Mattriks/CoupledFields.jl.… #8976

Merged
merged 3 commits into from
Apr 25, 2017
Merged

Register CoupledFields [https://github.com/Mattriks/CoupledFields.jl.… #8976

merged 3 commits into from
Apr 25, 2017

Conversation

Mattriks
Copy link
Contributor

See GiovineItalia/Gadfly.jl#791 (comment) for an example of what CoupledFields.jl can do!

@Mattriks Mattriks closed this Apr 24, 2017
@Mattriks Mattriks reopened this Apr 24, 2017
@@ -0,0 +1,3 @@
Compat 0.8.5
Copy link
Contributor

Choose a reason for hiding this comment

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

abstract type requires 0.17

@Mattriks
Copy link
Contributor Author

I found the small section on "Fixing Package Requirements", but the info is a bit sparse for me to know if I've done this correctly. I will now fix the Require file in the CoupledFields repo.

@tkelman tkelman closed this Apr 24, 2017
@tkelman tkelman reopened this Apr 24, 2017
@tkelman
Copy link
Contributor

tkelman commented Apr 24, 2017

are you using parametric aliases or Compat.Iterators? Why 0.18 instead of 0.17?

@Mattriks
Copy link
Contributor Author

Not currently, but the idea here is to add CoupledFields to Gadfly (see comment) so I made the Require file the same as is in the current master of Gadfly.

@tkelman
Copy link
Contributor

tkelman commented Apr 24, 2017

That's unnecessary, packages' dependencies are independent and should reflect what they use themselves.

@Mattriks
Copy link
Contributor Author

I thought about adding a parametric alias or a Compat.Iterators to CoupledFields 😃 , but instead I changed the require file to Compat 0.17.0.

@Mattriks Mattriks closed this Apr 24, 2017
@Mattriks Mattriks reopened this Apr 24, 2017
@tkelman tkelman merged commit a1eed2a into JuliaLang:metadata-v2 Apr 25, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants