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

Say goodbye to orchestra/testbench #305

Merged
merged 4 commits into from
Aug 25, 2019
Merged

Say goodbye to orchestra/testbench #305

merged 4 commits into from
Aug 25, 2019

Conversation

szepeviktor
Copy link
Collaborator

Closes #295

@nunomaduro
Copy link
Collaborator

Thanks for this @szepeviktor, I believe we should also adjust the readme. Can you write a section? Saying that for packages, the developer should install the package...

@szepeviktor
Copy link
Collaborator Author

In the Usage in Packages for Laravel section?

@nunomaduro
Copy link
Collaborator

Exactly buddy.

and empty lines after headers
@szepeviktor
Copy link
Collaborator Author

Done.

@szepeviktor szepeviktor merged commit 950cad7 into master Aug 25, 2019
@szepeviktor szepeviktor deleted the szepeviktor-patch-1 branch August 25, 2019 16:34
@nunomaduro
Copy link
Collaborator

@szepeviktor We should have targeted a develop branch for this. As will be shipped on the next major version.

@szepeviktor
Copy link
Collaborator Author

Do we have a develop branch?
https://github.com/nunomaduro/larastan/branches

@szepeviktor
Copy link
Collaborator Author

If I create one may I remove these commits from master?

@nunomaduro
Copy link
Collaborator

It's fine, give me some time to prepare a new version. I am looking a some issues that I can close easily before the major version.

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.

Improvement: remove orchestra dependency
2 participants