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

fix the *Tag.String method and add other useful methods #2

Closed
wants to merge 6 commits into from

Conversation

andeya
Copy link

@andeya andeya commented Aug 30, 2018

  • Fix the *Tag.String to solve double quotes escaping problem
  • Add .gitignore file
  • Add the *Tag.AddOptions method
  • Add the *Tag. DeleteOptions method

andeya and others added 6 commits August 30, 2018 16:00
Change-Id: Ida5b6810f6788d8ae66daddb73f0cf4eb00af02f
Change-Id: Iaf246838a4bce668d88b9063bb2a71b85da55653
Change-Id: I4c3e8922e09d76f1ec171ee8841e3c961c1cc6d9
Change-Id: I3b15cbaf175debe0145aaf42bd5980ac146e5b2f
Change-Id: I44c9f5542104cf53ddf4584547fb798964159d13
@fatih
Copy link
Owner

fatih commented Jul 10, 2019

Hi @henrylee2cn

Thanks for your PR. Can you please update the description on what you're trying to fix or improve? I see that you also added couple of new features, which I think shouldn't be part of this PR if you intend to do something else.

@andeya andeya changed the title quote tag value fix the *Tag.String method and add other useful methods Jul 10, 2019
@fatih fatih closed this in 3878f9f Sep 29, 2019
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