Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove unneeded and deprecated application commands builders. #2078

Merged
merged 2 commits into from
Jul 30, 2022

Conversation

mkrasnitski
Copy link
Collaborator

Since the builder rework landed, the CreateApplicationCommands builder has been made obsolete, as users can now simply pass around a Vec directly instead of through the wrapper builder. Additionally, removes the CreateApplicationCommandsPermissions builder and other functionality (as deprecated on current in #2077).

@github-actions github-actions bot added builder Related to the `builder` module. examples Related to Serenity's examples. model Related to the `model` module. labels Jul 29, 2022
@mkrasnitski mkrasnitski force-pushed the remove_plural_commands_builders branch from d844ea4 to c661cde Compare July 29, 2022 03:22
@arqunis arqunis added enhancement An improvement to Serenity. breaking change The public API is changed, resulting in miscompilations or unexpected new behaviour for users labels Jul 30, 2022
@arqunis arqunis merged commit daa15f6 into serenity-rs:next Jul 30, 2022
@mkrasnitski mkrasnitski deleted the remove_plural_commands_builders branch July 30, 2022 16:05
arqunis pushed a commit to arqunis/serenity that referenced this pull request Aug 21, 2022
arqunis pushed a commit to arqunis/serenity that referenced this pull request Sep 2, 2022
kangalio pushed a commit to kangalio/serenity that referenced this pull request Sep 11, 2022
mkrasnitski added a commit to mkrasnitski/serenity that referenced this pull request Oct 1, 2022
mkrasnitski added a commit to mkrasnitski/serenity that referenced this pull request Nov 7, 2022
mkrasnitski added a commit to mkrasnitski/serenity that referenced this pull request Feb 28, 2023
mkrasnitski added a commit to mkrasnitski/serenity that referenced this pull request May 18, 2023
mkrasnitski added a commit to mkrasnitski/serenity that referenced this pull request May 30, 2023
mkrasnitski added a commit to mkrasnitski/serenity that referenced this pull request Sep 21, 2023
mkrasnitski added a commit to mkrasnitski/serenity that referenced this pull request Oct 17, 2023
mkrasnitski added a commit to mkrasnitski/serenity that referenced this pull request Oct 24, 2023
arqunis pushed a commit to arqunis/serenity that referenced this pull request Oct 24, 2023
arqunis pushed a commit to arqunis/serenity that referenced this pull request Oct 24, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
breaking change The public API is changed, resulting in miscompilations or unexpected new behaviour for users builder Related to the `builder` module. enhancement An improvement to Serenity. examples Related to Serenity's examples. model Related to the `model` module.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants