Skip to content

Conversation

cubedhuang
Copy link
Contributor

About

This adds regex prefixes to the CommandClient and adds one in /test/class.ts. It addresses #306.

Status

  • These changes have been tested against Discord API or do not contain API change.
  • This PR includes only documentation changes, no code changes.
  • This PR introduces some Breaking changes.

Copy link
Member

@Helloyunho Helloyunho left a comment

Choose a reason for hiding this comment

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

lgtm, fix the lint issue and i think we can merge this

@cubedhuang
Copy link
Contributor Author

lgtm, fix the lint issue and i think we can merge this

i think this is an eslint issue because .startsWith should always return a boolean. the issue might be that the eslint version is using an old version of typescript which doesn't have better type guards, not sure how to fix

@Helloyunho
Copy link
Member

merging for now, I think I can fix the lint error

@Helloyunho Helloyunho merged commit 3c6f5f6 into harmonyland:main Aug 26, 2022
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