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

Changed from 'npm install' to 'yarn' in Jenkinsfile. Also minor changes. #83

Merged
merged 1 commit into from May 28, 2018

Conversation

@sharath-cliqz
Copy link

@sharath-cliqz sharath-cliqz commented May 28, 2018

No description provided.

@sharath-cliqz sharath-cliqz requested a review from ghostery/ghostery as a code owner May 28, 2018
@sammacbeth
Copy link
Contributor

@sammacbeth sammacbeth commented May 28, 2018

@sharath-cliqz please take upstream changes to Dockerfile too. An updated version of moab will be required for yarn compatibility.

@sharath-cliqz sharath-cliqz force-pushed the sharath-cliqz:android_browser branch from 5ce6053 to 1091a02 May 28, 2018
@sharath-cliqz
Copy link
Author

@sharath-cliqz sharath-cliqz commented May 28, 2018

@sammacbeth Thanks, I added the changes from the Dockerfile.

@sharath-cliqz sharath-cliqz force-pushed the sharath-cliqz:android_browser branch 2 times, most recently from f3997ff to be39ca3 May 28, 2018
@sharath-cliqz sharath-cliqz force-pushed the sharath-cliqz:android_browser branch from be39ca3 to fe11125 May 28, 2018
sh 'sed -i \'s/global.__DEV__/true/1\' node_modules/browser-core/build/core/console.js'
withGithubCredentials {
sh 'moab makezip'
withEnv(["NO_LINT=true"]) {

This comment has been minimized.

@luciancor

luciancor May 28, 2018
Contributor

@christophertino is this step always required?
Will merge this PR to keep the train moving but we probably need to double check this again before merging back to your branches (this and probably many more :) )

@luciancor luciancor merged commit 66ddef4 into ghostery:android_browser May 28, 2018
1 check failed
1 check failed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants