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.

Sign up for GitHub

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

Jump to bottom

Command line interface and miscellaneous refactoring #126

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

Command line interface and miscellaneous refactoring #126

wants to merge 31 commits into from

Conversation

apathor
Copy link
Contributor

@apathor apathor commented Feb 7, 2018 •
edited
Loading

WIP 😃

apathor added 30 commits January 7, 2018 11:39
@apathor
Start of dyntm command line tool.
15911d8
@apathor
A few more commands.
a54fe4b
@apathor
Roll logic into parent command. Recursively generate subcommands.
2d07360
@apathor
Comments and some record classes.
e6a9dc0
@apathor
Generate record type command classes.
05b010c
@apathor
Working record list, update, delete.
42fe34e
@apathor
Dry up TTL option. More sensible order for record list/get output.
8e742f9
@apathor
Refactor DNSRecord rdata method. Print rdata as JSON in record list c…
4e72cf0 
…ommands.
@apathor
Implement session token reuse.
9959d85
@apathor
Tiny cleanup.
0c62c7c
@apathor
Push _meta_update down. Saner auth failure error. Comments.
44e40a7
@apathor
Session refactor. Clean up retries, polling, and history.
6073deb
@apathor
Actual zone changeset support. Refactoring!
78c4423
@apathor
Support discarding pending changes too.
47a32db
@apathor
*** empty log message ***
c0a4c50
@apathor
Setup command line entry point for package.
f955351
@apathor
Fix passcmd choking on newlines.
2d68693
@apathor
Meet stylistic requirements.
2d73d22
@apathor
Refactor main dyntm command action.
a673ca7
@apathor
Shut up pycodestyle errors in sphix script.
39f346a
@apathor
Backoff exponentially on retry. Retry simultaneous tasks too.
7d0e20a
@apathor
Fix get_all_secondary_zones. Add secondary/primary zone list commands.
23e1129
@apathor
Make flake8 happy.
d1460d0
@apathor
Python 3 compatibility mostly?
e83d7d3
@apathor
Add pycodestyle check to CI target.
04e8ca9
@apathor
sed -i s/publsh/publish/ Makefile
1d56615
@apathor
Fix output of some commands.
6e2bc9c
@apathor
Fix misencoded password from passcmd under Python 3.
3752c6c
@apathor
Add dyntm command to create secondary zones.
4cc95cc
@apathor
Add redirect creation command.
351e32a
@derekargueta
Copy link

I'm not a maintainer/employee but maybe break up this refactor in smaller PRs so it's easier to review? +1,254 −309 is a lot of code to digest 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@apathor @derekargueta