Skip to content

Conversation

@cyrillkalita
Copy link
Contributor

  • Update dependency to support Laravel 8
  • Add CHANGELOG.md
  • Add composer scripts for testing and code fixing
  • Add ignore of .php_cs.cache
  • Apply PSR style formatting

- Update dependency to support Laravel 8
- Add [CHANGELOG.md](CHANGELOG.md)
- Add composer scripts for testing and code fixing
- Add ignore of .php_cs.cache
- Apply PSR style formatting
@nickfls
Copy link

nickfls commented Oct 9, 2020

@rtconner Rob, is there anything stopping the merge?
If i took too much liberty in running PSR fixes etc, I can create another PR with just a composer dependency update

@rtconner
Copy link
Owner

rtconner commented Oct 9, 2020

just lack of time on my part. sorry. I didn't even see this PR. what are all the white space changes?

@rtconner
Copy link
Owner

rtconner commented Oct 9, 2020

oh you probably did reformatting with phpstorm. I see.

@cyrillkalita
Copy link
Contributor Author

@rtconner I ran the code through phpcs.

@cyrillkalita
Copy link
Contributor Author

@rtconner do you think you may be able to review it any time soon?

@rtconner rtconner changed the base branch from laravel-7 to laravel-8 October 13, 2020 18:25
@rtconner rtconner merged commit b091270 into rtconner:laravel-8 Oct 13, 2020
Copy link
Owner

@rtconner rtconner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok this is released as version 3.1.0.
Thank you!

```php
class Article extends \Illuminate\Database\Eloquent\Model {
use \Conner\Likeable\LikeableTrait;
use \Conner\Likeable\Likeable;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wow, thanks for fixing this, I hadn't realized docs were wrong

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.

3 participants