Skip to content

Conversation

@mvrhov
Copy link
Contributor

@mvrhov mvrhov commented Dec 29, 2017

No description provided.

@Nyholm Nyholm self-requested a review December 29, 2017 12:06
@Nyholm
Copy link
Member

Nyholm commented Dec 29, 2017

Good. Im happy with using a trait like this.

Copy link
Member

@Nyholm Nyholm left a comment

Choose a reason for hiding this comment

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

Im not 100% happy with making Configuration mutable. But it will work for now =)

Thank you

@Nyholm Nyholm merged commit 396a441 into php-translation:master Dec 31, 2017
@mvrhov
Copy link
Contributor Author

mvrhov commented Dec 31, 2017

I know. But otherwise we would need to update the trait every time the configuration class would change-

@mvrhov mvrhov deleted the cmdBundle branch December 31, 2017 15:34
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