Skip to content
This repository has been archived by the owner on Aug 11, 2022. It is now read-only.

Fixes, support for TravisCI @php7.2 & Codecov #4

Merged
merged 3 commits into from
Aug 14, 2018
Merged

Fixes, support for TravisCI @php7.2 & Codecov #4

merged 3 commits into from
Aug 14, 2018

Conversation

rennokki
Copy link
Contributor

Hey. I love this package and i decided to fix a bit and add support for TravisCI @php7.2 & Codecov.

I usually use them, especially Codecov, and i have trouble when i want to start a new package: i just copy already done package's files and just re-write them a little bit so i can start a new one.

This is a good scaffolding. And i think it can be better.

Some of the devs are using StyleCI, and Codeclimate or Scrutinizer. Currently i am at the office and i cannot do really much, but in this evening i will add support for Scrutinizer & CircleCI.

Keep up the good work!

@rennokki
Copy link
Contributor Author

Check your StyleCI dashboard and create a fix PR, because i can't. :D

@Naoray
Copy link
Owner

Naoray commented Aug 14, 2018

Hey @rennokki,

thanks for your feedback :)!

I am pretty sure this package can be improved and I am happy to have you onboard helping me! I am currently already working on an improvement using Laravels GeneratorCommand. It cleans the code a bit and allows for easy extending the package funtionallity without making the MakePackage Command responsible to handle everything.

the stylecr thing will be fixed automatically when it is merged.

@Naoray Naoray merged commit b6c3ec0 into Naoray:master Aug 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants