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

Added command for adding domains to hosts file and some tests #3

Merged
merged 3 commits into from
Mar 19, 2020

Conversation

pacodes
Copy link
Contributor

@pacodes pacodes commented Mar 18, 2020

A variant of the add domain command.

@guumaster
Copy link
Owner

Thanks for this PR @pacodes!

I've added some commits changing a bit the commonAddOptions to have a simpler input for people who want to use it as a module (even tough I duplicated part of the struct).

Also added same functionality for domains to set, so you can set full content of a profile too.

Thanks for the tests! I should add lots more.

@guumaster guumaster marked this pull request as ready for review March 19, 2020 11:32
@guumaster guumaster merged commit ecbe34e into guumaster:master Mar 19, 2020
@guumaster
Copy link
Owner

@pacodes If you were planning to add more things, you are more than welcome. Just be sure to create an issue beforehand to discuss about it and avoid duplicating work.

Again many thanks for contributing 🥇 !

@pacodes
Copy link
Contributor Author

pacodes commented Mar 19, 2020

Ok @guumaster, thank you for your review and merge!

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.

2 participants