Skip to content

Conversation

@exussum12
Copy link
Collaborator

No description provided.

@exussum12 exussum12 force-pushed the optionToDisableColour branch from ddd68d6 to 77601d4 Compare October 30, 2021 22:19
@exussum12 exussum12 requested a review from sidz October 30, 2021 22:20
@exussum12
Copy link
Collaborator Author

Fixes #140

@exussum12
Copy link
Collaborator Author

@sidz thoughts on bumping Symphony console to 5.3 ? If your not a fan I can re write to have 2 options (--colour and --no colour separately)

Copy link
Collaborator

@sidz sidz left a comment

Choose a reason for hiding this comment

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

IMO we need to support at least 4.4 (LTS). phpmnd doesn't work well with 4.0 and php >= 7.4. But it will be right after we release 2.5. Just would like to merge #145 and we are okay to release 2.5.

Also I'd suggest to not include this feature in 2.5.

Regarding the feature itself:
I'm thinking about removing code highlighter at all and in this case we don't need an additional options.

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