Skip to content

Update composer scripts#1699

Merged
TomasVotruba merged 1 commit intorectorphp:masterfrom
ravanscafi:update-composer-scripts
Jul 6, 2019
Merged

Update composer scripts#1699
TomasVotruba merged 1 commit intorectorphp:masterfrom
ravanscafi:update-composer-scripts

Conversation

@ravanscafi
Copy link
Copy Markdown
Contributor

@ravanscafi ravanscafi commented Jul 5, 2019

There was a call to @update-docs but the script is called @docs.

Note: phpstan is failing on master too. I've fixed it on #1698

Comment thread composer.json Outdated
"check-cs": "ecs check bin packages src tests utils --ansi",
"fix-cs": [
"vendor/bin/ecs check bin packages src tests utils --fix --ansi",
"ecs check bin packages src tests utils --fix --ansi",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why this? It's problematic when copy-pasted to modify

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No problem, @TomasVotruba. I've updated the PR.

@TomasVotruba TomasVotruba merged commit 3baa863 into rectorphp:master Jul 6, 2019
@TomasVotruba
Copy link
Copy Markdown
Member

Thank you

@ravanscafi ravanscafi deleted the update-composer-scripts branch July 6, 2019 16:47
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.

2 participants