Skip to content
This repository has been archived by the owner on Jan 16, 2018. It is now read-only.

PHP 7 refactoring and improvements #53

Merged
merged 5 commits into from
Sep 21, 2016
Merged

PHP 7 refactoring and improvements #53

merged 5 commits into from
Sep 21, 2016

Conversation

sstok
Copy link
Contributor

@sstok sstok commented Sep 20, 2016

Q A
Bug Fix? yes
New Feature? yes
BC Breaks? yes
Deprecations? no
Fixed Tickets

Work in progress.

- Allow to modify after creation (each Datagrid is unique)
- Pass name when actually creating
- Strict types
- Less code complexity
@sstok
Copy link
Contributor Author

sstok commented Sep 21, 2016

That's strange, simplified_null_return is disabled but StyleCI doesn't disable this 🤔
But the local PHP-CS-Fixer bridge disabled it.

Edit: Mismatch between local configuration and configuration of the web interface.

@sstok sstok merged commit 320f227 into rollerworks-graveyard:master Sep 21, 2016
sstok added a commit that referenced this pull request Sep 21, 2016
This PR was merged into the master branch.

Discussion
----------

|Q            |A  |
|---          |---|
|Bug Fix?     |yes|
|New Feature? |yes|
|BC Breaks?   |yes|
|Deprecations?|no |
|Fixed Tickets|   |

Work in progress.

Commits
-------

a2d75f1 Add php strict typehint and return-type
234796a Refactor the DatagridBuilder
29f3f88 disable simplified_null_return for .styleci.yml
fdc77f9 Correct CS
320f227 Require newer version of PHPUnit
@sstok sstok deleted the php7-refactor branch September 21, 2016 09:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant