Skip to content
This repository has been archived by the owner on Jan 23, 2025. It is now read-only.

fix repo apply command #179

Merged
merged 5 commits into from
Oct 19, 2018
Merged

fix repo apply command #179

merged 5 commits into from
Oct 19, 2018

Conversation

jkwlui
Copy link
Member

@jkwlui jkwlui commented Oct 15, 2018

repo apply wasn't working for me out of the box, this shuold patch things up a little.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 15, 2018
await exec(`git clone ${cloneUrl} ${tmpDir.path}`);

let filesToUpdate;
try {
filesToUpdate = await updateCallback(tmpDir.path);
} catch (err) {
console.warn(
' callback function threw an exception, skipping this repository');
' callback function threw an exception, skipping this repository', err.stack);

This comment was marked as spam.

@jkwlui jkwlui requested a review from JustinBeckwith October 15, 2018 22:53
@@ -25,7 +25,18 @@ const commandLineUsage = require('command-line-usage');
import {updateRepo, UpdateRepoOptions} from './lib/update-repo';
import {question} from './lib/question';
import * as meow from 'meow';
const exec = pify(childProcess.exec);

This comment was marked as spam.

This comment was marked as spam.

@JustinBeckwith JustinBeckwith merged commit 4f0122e into master Oct 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants