Skip to content

Conversation

@phillipj
Copy link
Member

Reverting changes related to editing existing comments, going back to the previous behaviour of always creating new comments in PRs about CI / Jenkins runs.

This is done primarily due to two things:

  • It has broken node-core-utils' ability to check if CI's have been run
  • There's not collaborator consensus about editing comments is actually
    better than creating new comments

Future plans for bot PR comments going forward has to be discussed more thoroughly and ensured it doesn't break other automation tools used by collaborators.

Refs #228

/cc @nodejs/github-bot @richardlau@sam-github

Reverting changes related to editing existing comments, going back to the previous behaviour of always creating new comments in PRs about CI / Jenkins runs. This is done primarily due to two things: * It has broken node-core-utils' ability to check if CI's have been run * There's not collaborator consensus about editing comments is actually better than creating new comments Future plans for bot PR comments going forward has to be discussed more thorougly and ensured it doesn't break other automation tools used by collaborators. Refs nodejs#228
@phillipjphillipj merged commit 298e989 into nodejs:masterJun 15, 2019
@phillipjphillipj deleted the revert-edit-ci-comment branch June 15, 2019 13:22
Sign up for freeto 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

@phillipj@richardlau