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

Sets js_errors to false #8357

Merged

Conversation

filipefurtad0
Copy link
Contributor

What? Why?

Relates to #8356
I could not fix the specs which are the underlying cause for this, as described on the issue above, which remains open so we can address this seperatedly. This PR should turn the build back to green.

Setting js_errors to false should temporarily hide these errors which are breaking the build.

What should we test?

Green build.

Release notes

Sets js_errors to false on cuprite_setup.rb

Changelog Category: Technical changes

@achauve
Copy link
Collaborator

achauve commented Oct 14, 2021

If it means getting back a green build now and working on js_errors in a separate PR, it's just wonderful! ❤️

Sets js_errors to false

Sets headless to true
@filipefurtad0 filipefurtad0 force-pushed the remove_js_errors_on_system branch from 7e1655c to ef68791 Compare October 14, 2021 13:12
Copy link
Member

@mkllnk mkllnk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good call. Let's get the build back to green and then sort out the spec.

@mkllnk mkllnk merged commit feceff8 into openfoodfoundation:master Oct 15, 2021
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.

4 participants