Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add appveyor.yml #78

Merged
merged 1 commit into from Apr 30, 2016
Merged

add appveyor.yml #78

merged 1 commit into from Apr 30, 2016

Conversation

stefanpenner
Copy link
Collaborator

No description provided.

@stefanpenner stefanpenner mentioned this pull request Apr 28, 2016
5 tasks
@stefanpenner
Copy link
Collaborator Author

@ef4 / @asakusuma so it seems like the output of the files in-question are just different because of windows path segments. at-least that is was it seems.

We could:

  • generate the test-output on a per platform basis
  • test the behavior of the output rather then the input.
  • munge the output?

Thoughts?

@ef4
Copy link
Owner

ef4 commented Apr 29, 2016

I'd be in favor of behavioral testing instead. It's better anyway.

@stefanpenner
Copy link
Collaborator Author

stefanpenner commented Apr 30, 2016

Ok, i'll merge this anyways. It doesn't make stuff worse :P

I'll find some time this weekend to transition to behavioral tests

@stefanpenner stefanpenner merged commit 815328b into ef4:master Apr 30, 2016
@stefanpenner stefanpenner deleted the appveyor branch April 30, 2016 05:25
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.

None yet

2 participants