Skip to content
This repository was archived by the owner on Mar 27, 2025. It is now read-only.

Only add the comment for no id if the comment does not already exist - #28

Open
edhgoose wants to merge 1 commit into
ClearTax:masterfrom
edhgoose:feature-only-add-no-id-comment-once
Open

Only add the comment for no id if the comment does not already exist#28
edhgoose wants to merge 1 commit into
ClearTax:masterfrom
edhgoose:feature-only-add-no-id-comment-once

Conversation

@edhgoose

@edhgoose edhgoose commented Mar 4, 2020

Copy link
Copy Markdown
Contributor

Hey,

We have a few cases where having no id is sort of ok (temporary branches, hacking around, sharing an idea quickly), and at the moment every time we push to that branch, we get a new comment.

This stops the comment from being added if it already exists.

Thanks!

@edhgoose

Copy link
Copy Markdown
Contributor Author

Hey @maddhruv - I just tripped over this problem in a repository again, any chance of a merge? Happy to address any thoughts you have? Thanks!!

@edhgoose

Copy link
Copy Markdown
Contributor Author

(P.s. I've been using this branch in production since I added this PR - it's working great!)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant