Skip to content
This repository was archived by the owner on Jan 19, 2019. It is now read-only.

[FIX][no-unused-vars] add EnumMember #236

Merged
merged 3 commits into from
Dec 16, 2018
Merged

[FIX][no-unused-vars] add EnumMember #236

merged 3 commits into from
Dec 16, 2018

Conversation

mogafk
Copy link
Contributor

@mogafk mogafk commented Dec 16, 2018 •
edited by bradzacher
Loading

Fixes #235

@mogafk mogafk mentioned this pull request Dec 16, 2018
Closed
@j-f1
Copy link
Collaborator

j-f1 commented Dec 16, 2018

You might want to make sure the email you used to commit is added to your account so you get credit.

@armano2 armano2 mentioned this pull request Dec 16, 2018
Closed
bradzacher
bradzacher approved these changes Dec 16, 2018
View reviewed changes
Copy link
Owner

@bradzacher bradzacher left a comment

Choose a reason for hiding this comment

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

LGTM

@bradzacher bradzacher merged commit ae50541 into bradzacher:master Dec 16, 2018
@micaelmbagira
Copy link

@armano2 when do you plan to release this?

@bradzacher
Copy link
Owner

This is part of the upcoming 1.0.0 release, coming soon.
Please see the 1.0.0 issue for discussion.

@j-f1
Copy link
Collaborator

j-f1 commented Dec 20, 2018

(#151)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@bradzacher bradzacher bradzacher approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mogafk @j-f1 @micaelmbagira @bradzacher