Skip to content

Conversation

@lyrixx
Copy link
Member

@lyrixx lyrixx commented Oct 3, 2018

This is a bit longer, but this is much more explicite.
This is much easier to find all usage in a code base because the term is now unique

This is a bit longer, but this is much more explicite.
This, is much easier to find all usage in a code base because the term is now unique
@lyrixx lyrixx added the RFC label Oct 3, 2018
@lyrixx lyrixx requested a review from javiereguiluz October 3, 2018 09:24
Copy link
Member

@javiereguiluz javiereguiluz left a comment

Choose a reason for hiding this comment

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

I like this. Let's be more explicit!

@weaverryan
Copy link
Member

Great idea! I like this much better

@weaverryan weaverryan merged commit 5405fc9 into master Oct 4, 2018
weaverryan added a commit that referenced this pull request Oct 4, 2018
This PR was merged into the 1.0-dev branch.

Discussion
----------

[Voter] use ENTITY_ATTR instead of ATTR

This is a bit longer, but this is much more explicite.
This is much easier to find all usage in a code base because the term is now unique

Commits
-------

5405fc9 [Voter] use ENTITY_ATTR instead of ATTR
@lyrixx lyrixx deleted the lyrixx-patch-1 branch October 5, 2018 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants