Use New JuMP Macro API

[pulsipher]

This tries out jump-dev/JuMP.jl#3620 as a proof of concept.

cc @odow

[codecov]

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (ad13a33) 99.76% compared to head (f78eabe) 99.74%.

Files	Patch %	Lines
src/macros.jl	98.79%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #334      +/-   ##
==========================================
- Coverage   99.76%   99.74%   -0.02%     
==========================================
  Files          36       36              
  Lines        6680     6589      -91     
==========================================
- Hits         6664     6572      -92     
- Misses         16       17       +1     

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

[odow]

The 1.6 failure is our fault, because we don't have <0.0.1 here: https://github.com/jump-dev/JuMP.jl/blob/5b1a53f0a5adb7028ec2356f6239657a21475891/Project.toml#L24

[pulsipher]

The 1.6 failure is our fault, because we don't have <0.0.1 here: https://github.com/jump-dev/JuMP.jl/blob/5b1a53f0a5adb7028ec2356f6239657a21475891/Project.toml#L24

Ok, I'll wait for the fix then.

[odow]

Ok, I'll wait for the fix then.

It's just because you're using Pkg in the tests. This is a bug in Julia 1.6. When we tag a release of JuMP it'll work.

Edit: I guess it doesn't hurt: jump-dev/JuMP.jl#3626

[odow]

Hopefully fixed, so try re-running the tests.

[pulsipher]

Hopefully fixed, so try re-running the tests.

That seems to have fixed it, thanks!