Skip to content

[Calc]: Add support for reactions.#152

Merged
Vexed01 merged 5 commits intoVexed01:masterfrom
Evanroby:master
Sep 18, 2025
Merged

[Calc]: Add support for reactions.#152
Vexed01 merged 5 commits intoVexed01:masterfrom
Evanroby:master

Conversation

@Evanroby
Copy link
Contributor

@Evanroby Evanroby commented Sep 3, 2025

This PR is adding support for messages without command, with an opt-in feature using [p]calcset autocal <true/false>

Could you also make vexutils accept a __contributor__ part on your format_help for future? So users making PRs can get credits.

@Evanroby Evanroby requested a review from Vexed01 as a code owner September 3, 2025 18:55
@Vexed01
Copy link
Owner

Vexed01 commented Sep 17, 2025

Could you also make vexutils accept a __contributor__ part on your format_help for future? So users making PRs can get credits.

Done, please merge from master and add the __contributors__ attribute to the cog as you see prefer.

@Evanroby
Copy link
Contributor Author

Could you also make vexutils accept a __contributor__ part on your format_help for future? So users making PRs can get credits.

Done, please merge from master and add the __contributors__ attribute to the cog as you see prefer.

I think what you should have done is check if contributors exists, since now it’s going to display contributors even if theres none, right?

@Vexed01
Copy link
Owner

Vexed01 commented Sep 17, 2025 via email

@Evanroby
Copy link
Contributor Author

PR is good to go for me. lmk if you need code changes.

@Vexed01 Vexed01 merged commit beeb44b into Vexed01:master Sep 18, 2025
11 checks passed
@Vexed01
Copy link
Owner

Vexed01 commented Sep 18, 2025

Thanks for the PR!

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