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

fix: add shepherd to the list of supported init programs #354

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jbweston
Copy link

This makes it so that 'username@host' is not spuriously displayed on Guix System when not running in a container.

related: fixes #353

This makes it so that 'username@host' is not spuriously
displayed on Guix System.
@edouard-lopez
Copy link
Member

Thanks for the PR!

Please review the Acceptance Checks.
Could you add a test in tests/_pure_detect_container_by_pid_method.test.fish to cover your case?

@edouard-lopez edouard-lopez added the ⭐️ idea feature request or suggestion label Dec 9, 2024
@jbweston
Copy link
Author

@edouard-lopez I added a testcase and ran the tests according to your instructions against fish 3.3.1 (the example in the contributing guide) and 3.7.0 (the latest versions available on dockerhub?) and they all passed.

I read through the acceptance checks but most don't seem relevant for this contribution (i.e. this is not a feature with associated configuration variables etc.)

AFAICT I am in compliance with the checks, but let me know what you think!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⭐️ idea feature request or suggestion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

user@hostname is displayed on Guix System, even when not in a container
2 participants