Skip to content

/ mozjs

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

testing travis #53

Closed
wants to merge 4 commits into from
Closed

testing travis #53

wants to merge 4 commits into from

Conversation

@jdm
Copy link
Member

jdm commented Sep 21, 2015

Review on Reviewable
@jdm
testing travis
Loading status checks…
31a5fc9
@jdm jdm closed this Sep 21, 2015
@jdm jdm deleted the jdm-patch-1 branch Sep 21, 2015
@jdm jdm restored the jdm-patch-1 branch Sep 21, 2015
@jdm jdm reopened this Sep 21, 2015
Ms2ger added 2 commits Sep 22, 2015
@Ms2ger
Create .travis.yml
Loading status checks…
10c8185
@Ms2ger
Update build.rs
Loading status checks…
b24f86c
@Ms2ger
Copy link
Collaborator

Ms2ger commented Sep 22, 2015

@Manishearth can you please turn off appveyor?

@jdm travis passed
@jdm
Copy link
Member Author

jdm commented Sep 22, 2015

I suppose the real test here is to make a change that should break the build and ensure that happens :)
@Ms2ger
Break stuff
Loading status checks…
7cfd45c
@larsbergstrom
Copy link
Contributor

larsbergstrom commented Oct 7, 2015

@jdm @Ms2ger Is this ready to be merged, once you remove the "Break stuff" commit?
@jdm
Copy link
Member Author

jdm commented Oct 7, 2015

Closing in favour of #57.
@jdm jdm closed this Oct 7, 2015
@frewsxcv frewsxcv deleted the jdm-patch-1 branch Oct 7, 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
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
@jdm @Ms2ger @larsbergstrom
You can’t perform that action at this time.