Skip to content

Use lpsum instead of plain addition for constraints#64

Merged
meatballs merged 8 commits intomasterfrom
issue-56
May 9, 2017
Merged

Use lpsum instead of plain addition for constraints#64
meatballs merged 8 commits intomasterfrom
issue-56

Conversation

@meatballs
Copy link
Member

Fixes #56

@meatballs
Copy link
Member Author

Whoops. I committed a change to setup.cfg that I've been testing on this branch by accident.

@drvinceknight
Copy link
Collaborator

I've been down the road of getting pytest as a runner for the doctests: #28

I came to the conclusion that it wasn't the way to go:

  • I could tell travis specifically (not appveyor) to run them so that made appveyor happy (it looks like the order is OS dependent)
  • That doesn't work because travis doesn't get failures as a failed signal.

@meatballs
Copy link
Member Author

meatballs commented May 9, 2017

Pretty much the conclusion I'm coming to - just didn't intend to do it on this branch!

@meatballs
Copy link
Member Author

Oh good grief, it failed to commit the reversal!!! Agggghhhh......

@meatballs
Copy link
Member Author

coveralls.io is playing up. It's showing that there's no change in coverage at https://coveralls.io/builds/11432442 but not reporting it here

@drvinceknight
Copy link
Collaborator

Yeah I've been waiting on it. This looks good to me, feel free to merge 👍

@meatballs meatballs merged commit cfd7648 into master May 9, 2017
@meatballs meatballs deleted the issue-56 branch May 9, 2017 08:25
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 1cf6e71 on issue-56 into 4044583 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 2dff7b8 on issue-56 into 4044583 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 4fcb4a1 on issue-56 into 4044583 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 0354e62 on issue-56 into 4044583 on master.

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