Skip to content
This repository has been archived by the owner on Feb 11, 2021. It is now read-only.

ResultTypes minimum bound #37

Merged
merged 2 commits into from
Jun 5, 2018
Merged

ResultTypes minimum bound #37

merged 2 commits into from
Jun 5, 2018

Conversation

rofinn
Copy link
Member

@rofinn rofinn commented May 31, 2018

Specifically, the issue I was getting is:

ERROR: Unsatisfiable requirements detected for package ResultTypes:
...
├─version range [2.0.0,∞) required by package MyPkg, whose only allowed version is 0.2.0+:
│ └─version 0.2.0+ set by fixed requirement (package is checked out, dirty or pinned)
└─version range [0.1.0,2.0.0) required by package Dispatcher, whose allowed version range is [0.0.0-,∞):
...
The intersection of the requirements is empty.

I'm not sure why the default version range was [0.1.0,2.0.0) though :/

… use v2.0.0

```
ERROR: Unsatisfiable requirements detected for package ResultTypes:
...
├─version range [2.0.0,∞) required by package MyPkg, whose only allowed version is 0.2.0+:
│ └─version 0.2.0+ set by fixed requirement (package is checked out, dirty or pinned)
└─version range [0.1.0,2.0.0) required by package Dispatcher, whose allowed version range is [0.0.0-,∞):
The intersection of the requirements is empty.
```

I'm not sure why the default version range was [0.1.0,2.0.0) though :/
@rofinn rofinn changed the title Added a minimum bound for ResultTypes in order for another package to… ResultTypes minimum bound May 31, 2018
@rofinn rofinn requested a review from iamed2 May 31, 2018 17:52
@codecov
Copy link

codecov bot commented May 31, 2018

Codecov Report

Merging #37 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #37   +/-   ##
=======================================
  Coverage   82.29%   82.29%           
=======================================
  Files           4        4           
  Lines         305      305           
=======================================
  Hits          251      251           
  Misses         54       54

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 bc8b67d...8d93375. Read the comment docs.

test/runtests.jl Outdated
@@ -3,9 +3,12 @@ using ResultTypes
using Base.Test
using Memento
using IterTools
import Compat: @__MODULE__
using Compat
Copy link
Member

@iamed2 iamed2 May 31, 2018

Choose a reason for hiding this comment

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

Please revert this change (or change to using Compat: @__MODULE__)

@rofinn rofinn merged commit 23f4bb8 into master Jun 5, 2018
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.

None yet

2 participants