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

Enable debug assertions for SpiderMonkey in CI #19150

Closed
wants to merge 1 commit into from
Closed

Enable debug assertions for SpiderMonkey in CI #19150

wants to merge 1 commit into from

Conversation

@jdm
Copy link
Member

jdm commented Nov 8, 2017

This should catch more instances of API misuse.


This change is Reviewable

@highfive
Copy link

highfive commented Nov 8, 2017

Heads up! This PR modifies the following files:

@jdm
Copy link
Member Author

jdm commented Nov 8, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Nov 8, 2017

Trying commit 6449448 with merge af21bd1...

bors-servo added a commit that referenced this pull request Nov 8, 2017
Enable debug assertions for SpiderMonkey in CI

This should catch more instances of API misuse.
@bors-servo
Copy link
Contributor

bors-servo commented Nov 8, 2017

💔 Test failed - mac-rel-wpt3

@emilio
Copy link
Member

emilio commented Nov 8, 2017

r=me, if it passes

@jdm
Copy link
Member Author

jdm commented Nov 8, 2017

So many timeouts. This is not an improvement, unfortunately :(

@jdm jdm closed this Nov 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.