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
feat: add host module #286
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR @ahouts! Can't wait to get this into the trunk.
You've done a great job with most of this. There's just a few small styling/convention tweaks I'd like to do and a few questions I'd like to have answered just so that I understand some of the implications.
After that, we can get this baby merged!
This reverts commit 6af2cd9.
@chipbuster changes from the review are pushed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! 👍
Co-Authored-By: Matan Kushner <hello@matchai.me>
Thank you for your contribution, Andrew! 😄 @all-contributors Please add @ahouts for code, docs and test! |
I've put up a pull request to add @ahouts! 🎉 |
Thanks Andrew. I can't wait to use this. |
Description
Add a host module as requested by @chipbuster.
Displays the system hostname as provided by gethostname.
Motivation and Context
Closes #157
Types of changes
Screenshots (if appropriate):
How Has This Been Tested?
Checklist: