Skip to content

Conversation

@0x2b3bfa0
Copy link
Member

No description provided.

@0x2b3bfa0 0x2b3bfa0 requested a review from DavidGOrtega June 30, 2021 13:56
@0x2b3bfa0 0x2b3bfa0 temporarily deployed to internal June 30, 2021 13:56 Inactive
@0x2b3bfa0 0x2b3bfa0 removed the request for review from DavidGOrtega June 30, 2021 13:56
@0x2b3bfa0 0x2b3bfa0 temporarily deployed to internal June 30, 2021 13:56 Inactive
@0x2b3bfa0 0x2b3bfa0 self-assigned this Jun 30, 2021
@0x2b3bfa0 0x2b3bfa0 linked an issue Jun 30, 2021 that may be closed by this pull request
@0x2b3bfa0 0x2b3bfa0 changed the title Avoid depending on git for repo address inferrence Handle environments without Git with a helpful error message Jun 30, 2021
@0x2b3bfa0 0x2b3bfa0 closed this Jun 30, 2021
if (!repo) repo = gitRemoteUrl();
} catch (err) {
err.message +=
', you might need to manually specify the --repo option or install Git';
Copy link
Contributor

@casperdcl casperdcl Jul 1, 2021

Choose a reason for hiding this comment

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

super minor: maybe newline instead of ', '

Copy link
Member Author

Choose a reason for hiding this comment

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

@DavidGOrtega asked me to let him solve #618 by replacing the git command-line tool by a JavaScript library, so I have closed this pull request. Nevertheless, your comment is still on topic and useful. Can you please move it to #606?

Copy link
Member Author

Choose a reason for hiding this comment

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

Historical note: allegedly, this pull request was closed in favor of a then nonexistent #1003.

@0x2b3bfa0 0x2b3bfa0 deleted the avoid-depending-on-git-for-repo-address-inferrence branch July 2, 2021 14:20
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