Skip to content

Remove Codeapprove image once PR is approved. #65

Open

Description

@rockwotj

GitHub recently announced new options for controlling the commit message when PRs are merged. We use squash and merge, and the actual commits people make are mostly useless, so we've started defaulting to the Pull Request and description as the commit when squashed into master. The problem with that is that now we're getting the review image and label in our commits on master, which makes reading the commits that have happened very noisey.

Most commits on master end with:

<a data-ca-tag
    href="https://codeapprove.com/pr/<redacted>"><img
    src="https://codeapprove.com/external/github-tag-allbg.png" alt="Review
    on CodeApprove" /></a>

Which is a ton of noise. When a PR is approved in Code Approve could that badge be automatically removed? Or at least turn it into something more text friendly at the latest.

If you're unwilling to remove the link on approval, can you at least update the link to be markdown instead of HTML?

[![Review on CodeApprove](https://codeapprove.com/external/github-tag-allbg.png)](https://codeapprove.com/pr/<redacted>)

If you need the data-ca-tag to hide it with CSS on codeapprove.com I'd suggest changing your selector to:

img[alt="Review on CodeApprove"] {
  display: none;
}
/* or even add the src to be super explicit */
img[alt="Review on CodeApprove"][src="https://codeapprove.com/external/github-tag-allbg.png"] {
  display: none;
}

That will hide the image, and since the image is gone the link won't be clickable either.

Activity

  1. tstirrat commented on Sep 27, 2022

    @tstirrat

    I wonder if theres a way to have a CodeApprove "deployment" - a link to CA - that is easy to see and doesn't create an extra email, nor does it clog up the main PR description.

    Screen Shot 2022-09-27 at 9 34 14 AM

  2. samatcodeapprove commented on Sep 28, 2022

    @samatcodeapprove

    @rockwotj I love how you thought through all of my options haha. I will find a way to do at least one of the things you suggested!

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions