Skip to content
This repository has been archived by the owner on Nov 10, 2021. It is now read-only.

Ignore prefixes #92

Merged
merged 7 commits into from
Mar 18, 2021
Merged

Ignore prefixes #92

merged 7 commits into from
Mar 18, 2021

Conversation

EarlJr53
Copy link
Member

Ignores bot command prefixes when giving out XP. Closes #87

I decided for now that we only need to ignore prefixes, not channels.

@EarlJr53 EarlJr53 requested a review from i-am-gizm0 March 17, 2021 23:18
@EarlJr53 EarlJr53 self-assigned this Mar 17, 2021
@EarlJr53 EarlJr53 enabled auto-merge (squash) March 17, 2021 23:40
@EarlJr53 EarlJr53 added the enhancement New feature or request label Mar 18, 2021
@i-am-gizm0
Copy link
Contributor

Is this ready for review?

Copy link
Contributor

@i-am-gizm0 i-am-gizm0 left a comment

Choose a reason for hiding this comment

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

Looks good to me but it won't merge until there's another commit that has tests run on it. So commit some minor change that won't be formatted by Prettier and I can approve it and merge

@EarlJr53
Copy link
Member Author

Should be ready for review, yes. I can go find some minor change to make.

@i-am-gizm0
Copy link
Contributor

u g h. i'll approve but i have to re-enable override

i-am-gizm0
i-am-gizm0 previously approved these changes Mar 18, 2021
config.json.example Outdated Show resolved Hide resolved
@EarlJr53 EarlJr53 merged commit c906e33 into master Mar 18, 2021
@EarlJr53 EarlJr53 deleted the ignore-prefixes branch March 18, 2021 11:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Ignore messages in certain channels or with certain prefixes
3 participants