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

treehouses blocker (fixes #657) (fixes #157) #658

Merged
merged 44 commits into from
Feb 13, 2020
Merged

treehouses blocker (fixes #657) (fixes #157) #658

merged 44 commits into from
Feb 13, 2020

Conversation

LordJashin32
Copy link
Member

@LordJashin32 LordJashin32 commented Feb 1, 2020

Fixes #657
Fixes #159 possibly

  • Uses https://github.com/StevenBlack/hosts
  • blocker <0|1|2|3|4|max>
  • Blocks websites through editing of the /etc/hosts file
  • To test:
    Okay now it works for manual install. Still need to test on new image/npm version. Cheers.
    Run this to test:
git clone http://github.com/treehouses/cli
cd cli;
sudo npm i -g --unsafe-perm ./
sudo treehouses blocker
sudo treehouses blocker 1
sudo treehouses blocker
sudo treehouses blocker 3
sudo vim /etc/hosts

/etc/hosts should be changing whenever a new blocker level is set.

@LordJashin32 LordJashin32 self-assigned this Feb 1, 2020
Copy link
Member

@hiroTochigi hiroTochigi left a comment

Choose a reason for hiding this comment

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

Do I need to use the latest image to test the branch?
Or, do I need to add something to test the branch?

@LordJashin32
Copy link
Member Author

Do I need to use the latest image to test the branch?
Or, do I need to add something to test the branch?

We'll have to download the image to test it. But for now I have not tested if its working. So gonna put this on hold for now.

@LordJashin32
Copy link
Member Author

Okay now it works for manual install. Still need to test on new image/npm version. Cheers.
Run this to test:

git clone http://github.com/treehouses/cli
cd cli;
sudo npm i -g --unsafe-perm ./
sudo treehouses blocker
sudo treehouses blocker 1
sudo treehouses blocker
sudo treehouses blocker 3
sudo vim /etc/hosts

/etc/hosts should be changing whenever a new blocker level is set.

@dogi dogi changed the title add blocker command treehouses blocker (fixes #657)(fixes #157) Feb 13, 2020
@dogi dogi changed the title treehouses blocker (fixes #657)(fixes #157) treehouses blocker (fixes #657) (fixes #157) Feb 13, 2020
@dogi dogi merged commit 4fcfd27 into master Feb 13, 2020
@dogi dogi deleted the add-hosts-blocker branch February 13, 2020 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
4 participants