Skip to content

CLI refactoring #22

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
Merged
javibravo merged 30 commits into from
May 8, 2017
Merged

CLI refactoring #22

javibravo merged 30 commits into from
May 8, 2017

Conversation

@javibravo
Copy link
Contributor

@javibravo javibravo commented May 3, 2017

Refactoring LambdaCron CLI:

  • Abstract CLI commands to classes and divide responsibilities
  • Clean CLI config management.
  • Improve testing

javibravo added 30 commits April 23, 2017 10:54
@javibravo
Abstract commands to classes: ValidateCommand
e93ddac
@javibravo
Add Copyright in new files
2fb1a6f
@javibravo
Use ValidateCommand class
e8c485b
@javibravo
Handle exit in cli_tool.py
918af38
@javibravo
Remove old test for validate command
5c397f6
@javibravo
Add command abstraction for Create
40b9bfc
@javibravo
Update tests for update command
f3ec400
@javibravo
Add delete command
601af6d
@javibravo
Refactor CloudFormation commands
f6a7365
@javibravo
Refactor start and stop commands
c97adbf
@javibravo
Add Upload Tasks command
c991cef
@javibravo
Refactor exec command
bd842f8
@javibravo
Reorg commands
7a2ef6c
@javibravo
Reorg tests
a76b00f
@javibravo
Refactor invoke command
df9c750
@javibravo
Refactoring Config
5e2860a
@javibravo
Refactor CLI Config
ba64910
@javibravo
Refactor CliTool class
44ac578
@javibravo
Fix CliTool
ebc0a7e
@javibravo
Update command summary to print
e3806dd
@javibravo
Fix read config with default environment settings
d6f3234
@javibravo
Re-org test files
d001136
@javibravo
Clean tests
46bb09a
@javibravo
Fix and test execution for Inovke command
21747b0
@javibravo
Complete AWS profile tests and clean tests names
fb8825b
@javibravo
Move code to cloudoformation commands
9e00b34
@javibravo
Move code to CloudFormation command
7ed8ba5
@javibravo
Update Readme
89b7299
@javibravo
Inject config file to CLI Tool
cc076fb
@javibravo
Fix to include template in CloudFormation commands
0712326
@javibravo javibravo merged commit 8444186 into master May 8, 2017
@javibravo javibravo deleted the commands_abstraction branch May 8, 2017 08:13
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

@javibravo