Skip to content
This repository has been archived by the owner on May 20, 2024. It is now read-only.

[NEEDS TESTING] Remove thank you message #1103

Merged
merged 1 commit into from
Dec 3, 2018

Conversation

gundalow
Copy link
Collaborator

As this was posted as it's own comment, it was found to be a bit spammy.

The long term fix for this is to have a single message when Issues/PRs
are created, see
#1081 (comment)

As this was posted as it's own comment, it was found to be a bit spammy.

The long term fix for this is to have a *single* message when Issues/PRs
are created, see
ansible#1081 (comment)
@gundalow
Copy link
Collaborator Author

I've done zero testing on this

@codecov
Copy link

codecov bot commented Nov 27, 2018

Codecov Report

Merging #1103 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1103      +/-   ##
==========================================
+ Coverage   31.45%   31.47%   +0.02%     
==========================================
  Files          47       47              
  Lines        9197     9190       -7     
==========================================
  Hits         2892     2892              
+ Misses       6305     6298       -7

@gundalow gundalow changed the title Remove thank you message [NEEDS TESTING] Remove thank you message Nov 27, 2018
Copy link
Member

@webknjaz webknjaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks ok

@gundalow gundalow merged commit c93eed6 into ansible:master Dec 3, 2018
@gundalow gundalow deleted the no-thanks branch December 3, 2018 15:17
@gundalow
Copy link
Collaborator Author

gundalow commented Dec 3, 2018

Confirmed this fixed it

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.

None yet

2 participants