Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tags refactor #1098

Merged
merged 2 commits into from
Feb 14, 2019
Merged

Tags refactor #1098

merged 2 commits into from
Feb 14, 2019

Conversation

randy3k
Copy link
Collaborator

@randy3k randy3k commented Feb 3, 2019

as discussed in #998, we are going to revert the previous commit which changes tags to tag but only keeping the user command.

@randy3k
Copy link
Collaborator Author

randy3k commented Feb 11, 2019

@stoivo
how does it look? I am cc'ing you because you merged the original commit.

@randy3k
Copy link
Collaborator Author

randy3k commented Feb 14, 2019

@kaste Is this what you anticipated?

I think @stoivo and @asfaltboy may not be free.

@kaste
Copy link
Collaborator

kaste commented Feb 14, 2019

LGTM 👍

@asfaltboy
Copy link
Member

Looks fine to me btw, I am all for consistency

@randy3k randy3k merged commit eeb4600 into dev Feb 14, 2019
@randy3k randy3k deleted the tags_refactor branch February 25, 2019 04:35
@stoivo
Copy link
Member

stoivo commented Mar 5, 2019

Yes I have been away.

@randy3k
Copy link
Collaborator Author

randy3k commented Mar 5, 2019

Welcome back.

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.

None yet

4 participants