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

Refactoring application framework #13 #15

Merged
merged 5 commits into from
Aug 25, 2017
Merged

Refactoring application framework #13 #15

merged 5 commits into from
Aug 25, 2017

Conversation

nntoan
Copy link
Owner

@nntoan nntoan commented Aug 25, 2017

This PR cover all stuffs below:

  • Refactoring application framework

  • Avoid code duplication in all exists commands by adding an abstract logic

  • Remove redundant codes

  • Fix few SensioLabInsight issues

  • Remove PHP 7.1 in Travis configuration, added Slack notifications

  • Coveraged some test cases in Gojira\Framework

- Update Travis CI and PHP Unit config files
- Move Phrase and Serializer to Gojira\Framework
- Added Encryption, Math and ObjectManager to Gojira\Framework
- Added serveral test cases for Gojira\Framework
- Update all commands to avoid code duplication (better abstract logic)
- Update endpoints and responses, better for designs
- Added new command ListInProgress (issue:list:in-progress)
- Move Gojira\Api\Configuration to Gojira\Framework\App\Configuration
- Move Gojira\Api\Data to Gojira\Framework\Data
- Move TableInterface to Gojira\Framework\App\Console
- Added new available issues statuses to default option
- Removed redundant 'use' statements
- Fixed incorrect 'use' statements of few commands
- Update .gitignore for /docs/ dir (ApiGen)
- Added new ProgressBar provider as console component
@nntoan nntoan added this to the August 2017 milestone Aug 25, 2017
@nntoan nntoan self-assigned this Aug 25, 2017
@coveralls
Copy link

coveralls commented Aug 25, 2017

Coverage Status

Coverage increased (+11.4%) to 14.334% when pulling 5e3e64d on gorira into a7df583 on master.

@nntoan nntoan merged commit 69cda4f into master Aug 25, 2017
@nntoan nntoan added this to TODO in branch [gorira] Aug 25, 2017
@nntoan nntoan moved this from TODO to Done in branch [gorira] Aug 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants