Skip to content

Merge fixing-all branch to dev #24

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

Merged
merged 2 commits into from
May 21, 2019
Merged

Merge fixing-all branch to dev #24

merged 2 commits into from
May 21, 2019

Conversation

andreacivita
Copy link
Owner

Fixed method all (finally)

@codecov
Copy link

codecov bot commented May 21, 2019

Codecov Report

Merging #24 into dev will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##                dev      #24   +/-   ##
=========================================
  Coverage     90.62%   90.62%           
  Complexity       21       21           
=========================================
  Files             2        2           
  Lines            64       64           
=========================================
  Hits             58       58           
  Misses            6        6

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9d2c628...60598fe. Read the comment docs.

@andreacivita andreacivita merged commit 66f674c into dev May 21, 2019
@andreacivita andreacivita deleted the fixing-all branch May 21, 2019 09:11
andreacivita added a commit that referenced this pull request May 21, 2019
* Change model folder name

* Add request comments

* Generate resource

* Generate resource controller

* CHange model namespace

* Fix generated message

* Create test in correct folder and namespace

* Generate factory

* Fix variable naming

* Test refactoring

* Fix factory path

* Fix test template

* Overwrite test instead append

* Revert "Merge pull request #7 from bastianjoel/master"

This reverts commit ecfaa98, reversing
changes made to 275df6f.

* Revert "Merge pull request #7 from bastianjoel/master"

This reverts commit ecfaa98, reversing
changes made to 275df6f.

* Update README.md

* Complete refactor (#12)

* Create CONTRIBUTING.MD

* Use laravel filesystem

* Fix cs

* Update README.md

* Fix bastianjoel pr errors

Fixing truble into pull request

* switched to STR

str_plural

* Removed unused dependencies

* Fix timestamp not working

* Refactoring PHPDoc & Added Str dependency

Refactored PHPDoc and added dependency injection for \Illuminate\Support\String

* Some style fix

* Refactoring ApiCrudGenerator class

Refactoring ApiCrudGenerator class for keeping number of methods under 10.
This should help in fixing.
Added Stub Class (Helper for parse & get stub)
Added Generator Class (Helper for generating files)

* Missing str dependency

* Updated readme

* Fix readme authors

* Moved core class into Core

* Fixed dependencies

* Added unit tests

* Added stub test and refactor laravel helpers

* Deleted laravel_helpers

* Fix generator

* Another fix

* Added test for generator

* Added travis support

* Fix

* Fix travis

* Passport integration (#16)

* Fix test & added test secure

* Added passport support

* Fix styling

* Secure routes

* Fix testSecureRoot

* Codecov + travis

* Delete CODE_OF_CONDUCT.md

* Fixed deprecated touch() call

* Added Dependency Injection

Refactored file with dependency injection for DB & Schema Facades

* Fix documentation

* Testing fix

* Fixed --all bug

* Delete LICENSE

* Merge fixing-all branch to dev (#24)

* Fixed all method

* Removed $db support instance
andreacivita added a commit that referenced this pull request Mar 13, 2021
* Change model folder name

* Add request comments

* Generate resource

* Generate resource controller

* CHange model namespace

* Fix generated message

* Create test in correct folder and namespace

* Generate factory

* Fix variable naming

* Test refactoring

* Fix factory path

* Fix test template

* Overwrite test instead append

* Revert "Merge pull request #7 from bastianjoel/master"

This reverts commit ecfaa98, reversing
changes made to 275df6f.

* Revert "Merge pull request #7 from bastianjoel/master"

This reverts commit ecfaa98, reversing
changes made to 275df6f.

* Update README.md

* Complete refactor (#12)

* Create CONTRIBUTING.MD

* Use laravel filesystem

* Fix cs

* Update README.md

* Fix bastianjoel pr errors

Fixing truble into pull request

* switched to STR

str_plural

* Removed unused dependencies

* Fix timestamp not working

* Refactoring PHPDoc & Added Str dependency

Refactored PHPDoc and added dependency injection for \Illuminate\Support\String

* Some style fix

* Refactoring ApiCrudGenerator class

Refactoring ApiCrudGenerator class for keeping number of methods under 10.
This should help in fixing.
Added Stub Class (Helper for parse & get stub)
Added Generator Class (Helper for generating files)

* Missing str dependency

* Updated readme

* Fix readme authors

* Moved core class into Core

* Fixed dependencies

* Added unit tests

* Added stub test and refactor laravel helpers

* Deleted laravel_helpers

* Fix generator

* Another fix

* Added test for generator

* Added travis support

* Fix

* Fix travis

* Passport integration (#16)

* Fix test & added test secure

* Added passport support

* Fix styling

* Secure routes

* Fix testSecureRoot

* Codecov + travis

* Delete CODE_OF_CONDUCT.md

* Fixed deprecated touch() call

* Added Dependency Injection

Refactored file with dependency injection for DB & Schema Facades

* Fix documentation

* Testing fix

* Fixed --all bug

* Delete LICENSE

* Merge fixing-all branch to dev (#24)

* Fixed all method

* Removed $db support instance

* Update to laravel 8

* Update .travis.yml

* Update

* Upgrade dependencies

* Laravel v8 supports

Supports also for php8 && phpunit 8

Co-authored-by: Bastian Rihm <[email protected]>
Co-authored-by: restyled-io[bot] <32688539+restyled-io[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant