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

style: cleanup command usage #85

Merged
merged 10 commits into from
Feb 4, 2021

Conversation

dmccaffery
Copy link
Member

No description provided.

NOTE:

The `docker-credential-gcr` helper should not be used on workstations
where gcloud is installed. Users should use the following instead:

```sh
gcloud auth configure-docker
```
NOTE:

This will only work on more recent distributions for WSL but will
dramatically improve performance of the prompt in git paths that are
mounted from the windows file system. The use of the windows version
of git only applies to windows mount path. The local (linux) version of
git is used otherwise.
@dmccaffery dmccaffery requested a review from sjk07 as a code owner January 11, 2021 19:54
@dmccaffery dmccaffery enabled auto-merge (rebase) January 11, 2021 19:56
@dmccaffery dmccaffery self-assigned this Jan 11, 2021
@patrickserrano
Copy link
Contributor

@dmccaffery tests for macOS are failing and it looks like there's no output?

src/sh/scripts/add-var Show resolved Hide resolved
src/sh/scripts/use-shell Show resolved Hide resolved
export GPG_TTY
export SSH_AUTH_SOCK

# launch gpg-agent using the expected tty and sock
Copy link
Contributor

Choose a reason for hiding this comment

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

@dmccaffery looks like the test for this script is failing on Debian 9.

Copy link
Contributor

Choose a reason for hiding this comment

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

@dmccaffery I ran the test on a new Debian instance and was able to execute the tests without issue. Do you want to chalk this up to CI flakiness?

@dmccaffery
Copy link
Member Author

@dmccaffery tests for macOS are failing and it looks like there's no output?

Yeah; they never queued. Pretty sure it was just a fluke, but the other failed builds kept me from requeueing for now.

@dmccaffery dmccaffery merged commit 3de0106 into automotiveMastermind:next Feb 4, 2021
@dmccaffery
Copy link
Member Author

🎉 This PR is included in version 8.3.0-next.4 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants