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

add flag clear_tag_comment to clear the tag comment #55

Merged
merged 1 commit into from Aug 12, 2022

Conversation

superqtqt
Copy link
Contributor

No description provided.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 85

  • 31 of 35 (88.57%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+1.6%) to 73.585%

Changes Missing Coverage Covered Lines Changed/Added Lines %
main.go 0 4 0.0%
Totals Coverage Status
Change from base Build 84: 1.6%
Covered Lines: 156
Relevant Lines: 212

💛 - Coveralls

@lrstanley
Copy link
Collaborator

lrstanley commented Aug 11, 2022

Hello! Could you further explain the purpose/feature addition of the PR, and what problem it's trying to resolve? Is it similar to this feature request?

@lrstanley lrstanley self-assigned this Aug 11, 2022
@superqtqt
Copy link
Contributor Author

Hello! Could you further explain the purpose/feature addition of the PR, and what problem it's trying to resolve? Is it similar to this feature request?

yes, I just want to implement the feature. #48

@lrstanley lrstanley merged commit 627bcea into favadi:master Aug 12, 2022
@lrstanley
Copy link
Collaborator

@superqtqt it has been merged -- I've made some minor adjustments, including renaming the flag to -remote_tag_comment and some of the variables to be a bit more consistent and clear.

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

3 participants