Skip to content

Disable good food for the summer#160

Merged
c0unts merged 3 commits intomainfrom
5731la-patch-1
May 13, 2025
Merged

Disable good food for the summer#160
c0unts merged 3 commits intomainfrom
5731la-patch-1

Conversation

@5731la
Copy link
Copy Markdown
Contributor

@5731la 5731la commented May 13, 2025

he will be missed

@5731la 5731la requested a review from a team as a code owner May 13, 2025 16:14
Copy link
Copy Markdown
Contributor

@c0unts c0unts left a comment

Choose a reason for hiding this comment

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

Please comment out line #73 in commands/enabled.go as well then it should be good to go.

Go gently into this deep sleep
@5731la 5731la requested a review from c0unts May 13, 2025 16:32
@c0unts
Copy link
Copy Markdown
Contributor

c0unts commented May 13, 2025

Can you listen to the code checks as well

Copy link
Copy Markdown
Contributor

@c0unts c0unts left a comment

Choose a reason for hiding this comment

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

Code checks

@5731la
Copy link
Copy Markdown
Contributor Author

5731la commented May 13, 2025

Code checks

Code checks are because the function is unused, do we want to comment out the entire sendGoodFoodPing function? It's a little messy but I can't think of a better way

@c0unts
Copy link
Copy Markdown
Contributor

c0unts commented May 13, 2025

remove the first commit, keep the second commit

@5731la 5731la requested a review from c0unts May 13, 2025 17:31
Copy link
Copy Markdown
Contributor

@c0unts c0unts left a comment

Choose a reason for hiding this comment

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

lgtm, will be re-deployed before tomorrow

@c0unts c0unts merged commit 3584c1e into main May 13, 2025
4 checks passed
Sign up for free to 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