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.

Sign up for GitHub

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

Jump to bottom

adding travis-ci #3

Closed
wants to merge 1 commit into from
Closed

adding travis-ci #3

wants to merge 1 commit into from

Conversation

countdigi
Copy link
Contributor

Here is a pull request that should set up travis-ci - I had to delete dt_003.data because it does not generate correctly. After this we either need you to set up travis-ci and/or give me access to the repo. Talk to you this afternoon. Thanks, Kevin

@piratical
Copy link
Owner

Hi, Kevin,

Yep, I added you as a collaborator on GitHub. This will allow you to push
the travis-ci stuff into the main Madeline repo. Let's discuss this
afternoon what this should look like. In particular, the output .xml files
should be compared against a set of .xml files generated by the latest
version of Madeline2, inter alia ...

Best — Ed

On Thu, Oct 8, 2015 at 5:12 AM, Kevin Counts notifications@github.com
wrote:

Here is a pull request that should set up travis-ci - I had to delete
dt_003.data because it does not generate correctly. After this we either
need you to set up travis-ci and/or give me access to the repo. Talk to you

this afternoon. Thanks, Kevin

You can view, comment on, or merge this pull request online at:

#3
Commit Summary

  • adding generation and comparison of output in bin/test-madeline for
    travis-ci

File Changes

Patch Links:

—
Reply to this email directly or view it on GitHub
#3.

@countdigi
Copy link
Contributor Author

Sounds good - so here is the script that is doing the comparison - its
comparing testdata/output against tmp/output which is where we generate the
SVG .xml files.

https://raw.githubusercontent.com/USF-HII/Madeline_2.0_PDE/master/bin/test-madeline

Needs a little refactoring but its working well for a first go.

On Thu, Oct 8, 2015 at 12:03 PM, Ed notifications@github.com wrote:

Hi, Kevin,

Yep, I added you as a collaborator on GitHub. This will allow you to push
the travis-ci stuff into the main Madeline repo. Let's discuss this
afternoon what this should look like. In particular, the output .xml files
should be compared against a set of .xml files generated by the latest
version of Madeline2, inter alia ...

Best — Ed

On Thu, Oct 8, 2015 at 5:12 AM, Kevin Counts notifications@github.com
wrote:

Here is a pull request that should set up travis-ci - I had to delete
dt_003.data because it does not generate correctly. After this we either
need you to set up travis-ci and/or give me access to the repo. Talk to
you

this afternoon. Thanks, Kevin

You can view, comment on, or merge this pull request online at:

#3
Commit Summary

  • adding generation and comparison of output in bin/test-madeline for
    travis-ci

File Changes

Patch Links:

—
Reply to this email directly or view it on GitHub
#3.

—
Reply to this email directly or view it on GitHub
#3 (comment)
.

Kevin Counts counts@digicat.org

@countdigi
Copy link
Contributor Author

scrapping this and going directly against main repo now that I am a collaborator

@countdigi countdigi closed this Oct 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@countdigi @piratical