Skip to content

Added CLI interface and did a little clean up.#4

Open
lwahlmeier wants to merge 3 commits intoStanback:masterfrom
lwahlmeier:master
Open

Added CLI interface and did a little clean up.#4
lwahlmeier wants to merge 3 commits intoStanback:masterfrom
lwahlmeier:master

Conversation

@lwahlmeier
Copy link

Added CLI ability because I wanted to run this from cron w/o using a web server. I also cleaned up some of the methods that where taking parameters they already had access to through the classes _config variable.

…s the same process, also updated some methods to depend more on local config instead of passed arguments. Added ability to run on the commandline
@Stanback
Copy link
Owner

Thanks for the pull request - this looks good.
I'll test it over the weekend and merge.

@snicker
Copy link

snicker commented Apr 22, 2014

This is working perfectly for me. thanks @lwahlmeier, makes the demise of my old dyndns account less of a bummer

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

Comments