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.

Sign up for GitHub

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

Jump to bottom

Strip references to seasonalbot chat #366

Merged
merged 1 commit into from Mar 3, 2020
Merged

Strip references to seasonalbot chat #366

merged 1 commit into from Mar 3, 2020

Conversation

sco1
Copy link
Contributor

@sco1 sco1 commented Mar 1, 2020

Redirect any output to seasonalbot commands, where relevant.

@sco1
Strip references to seasonalbot chat
10b63ed 
Redirect any output to seasonalbot commands, where relevant
@sco1 sco1 requested a review from a team as a code owner March 1, 2020 23:58
@sco1 sco1 requested review from eivl and jerbob March 1, 2020 23:58
SebastiaanZ
SebastiaanZ approved these changes Mar 2, 2020
View reviewed changes
Copy link
Member

@SebastiaanZ SebastiaanZ left a comment

Choose a reason for hiding this comment

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

I've had a look around the files as well and I think you've got them all.

Once this is merged, I'll remove the references from salt/pillar as well.

@SebastiaanZ SebastiaanZ added area: backend Related to internal functionality and utilities priority: 1 - high High Priority status: needs review Author is waiting for someone to review and approve labels Mar 2, 2020
eivl
eivl approved these changes Mar 3, 2020
View reviewed changes
Copy link
Contributor

@eivl eivl left a comment

Choose a reason for hiding this comment

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

I found the same amount of changes as this PR when I searched.

@eivl eivl merged commit 1fbf3ae into master Mar 3, 2020
@eivl eivl deleted the tidy-seasonal-channels branch March 3, 2020 06:58
@Xithrius Xithrius removed the status: needs review Author is waiting for someone to review and approve label Nov 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eivl eivl eivl approved these changes

@SebastiaanZ SebastiaanZ SebastiaanZ approved these changes

@jerbob jerbob Awaiting requested review from jerbob

Assignees
No one assigned
Labels
area: backend Related to internal functionality and utilities priority: 1 - high High Priority
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@sco1 @eivl @SebastiaanZ @Xithrius