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

Do not show hostname when running in GitHub Codespaces #604

Merged
merged 2 commits into from
Aug 15, 2021

Conversation

MarshallOfSound
Copy link
Contributor

When running in GitHub Codespaces pure thinks it's running inside a container (correctly) but we don't need the codespaces hostname when using the integrated terminal.

Tested this change using my dotfiles and the default codespaces image.

@sindresorhus sindresorhus changed the title do not show hostname when running in Codespaces Do not show hostname when running in GitHub Codespaces Aug 15, 2021
@mafredri mafredri merged commit 98a57d9 into sindresorhus:main Aug 15, 2021
@mafredri
Copy link
Collaborator

Thanks for the PR!

@MarshallOfSound MarshallOfSound deleted the patch-1 branch August 16, 2021 00:43
wezzynl added a commit to wezzynl/pure that referenced this pull request Sep 7, 2021
* upstream/main:
  Remove $ prompts from install commands (sindresorhus#607)
  1.17.2
  Improve Apple Silicon compatibility (sindresorhus#606)
  1.17.1
  Do not show hostname when running in GitHub Codespaces (sindresorhus#604)
  Fix version bumping script (sindresorhus#602)
  Add another sponsor to the readme
  1.17.0
  Add nix shell support (sindresorhus#599)
  Add Homebrew install instructions (sindresorhus#591)
  Update info about Fish port (sindresorhus#586)
  Remove antigen and antibody from instruction (sindresorhus#581)
  Update a link
kutsan pushed a commit to kutsan/pure that referenced this pull request Jun 19, 2023
)

Co-authored-by: Sindre Sorhus <sindresorhus@gmail.com>
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.

3 participants