Skip to content

Conversation

@jeremyharris
Copy link
Member

I changed the directive to csv-table as it's much easier to maintain, although the source isn't as easy to read. csv-table comes with the caveat that the number of columns per row isn't immediately visible nor enforced. If that's a deal killer we can close this PR.

Copy link
Member

@markstory markstory left a comment

Choose a reason for hiding this comment

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

This will be much easier to work with in the future.

@markstory markstory self-assigned this Aug 1, 2017
@markstory markstory merged commit 01d6971 into cakephp:3.0 Aug 2, 2017
@jeremyharris jeremyharris deleted the alphabetize-collection-methods branch August 2, 2017 01:34
@HavokInspiration HavokInspiration added this to the 3.x milestone Aug 2, 2017
HavokInspiration added a commit that referenced this pull request Aug 2, 2017
antograssiot added a commit that referenced this pull request Aug 2, 2017
okinaka added a commit to okinaka/docs that referenced this pull request Aug 6, 2017
@okinaka okinaka mentioned this pull request Aug 6, 2017
okinaka added a commit to okinaka/docs that referenced this pull request Aug 7, 2017
okinaka added a commit to okinaka/docs that referenced this pull request Aug 9, 2017
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.

4 participants