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

[2.0] Refactor Tapestry with regards to cli application. #280

Open
carbontwelve opened this issue Jan 2, 2018 · 0 comments
Open

[2.0] Refactor Tapestry with regards to cli application. #280

carbontwelve opened this issue Jan 2, 2018 · 0 comments
Labels
refactoring
Milestone
2.0.0

Comments

@carbontwelve
Copy link
Member

The Tapestry class is both the Container and application, while the cli Application is separate. This feels at odds to how the program actually works.

Tapestry should be the cli application class, with the object currently called Tapestry being renamed to what it actually is: Container.

NOTE: This will introduce breaking changes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
refactoring
Projects
None yet
Milestone
2.0.0
Development

No branches or pull requests
1 participant
@carbontwelve