Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

[Exiled::Events] Added new commands for installing plugins, changing or printing config values #2389

Merged
merged 17 commits into from
Jun 23, 2024
Merged

[Exiled::Events] Added new commands for installing plugins, changing or printing config values #2389

merged 17 commits into from
Jun 23, 2024

Conversation

VALERA771
Copy link
Contributor

[Exiled::API] Added extension

StringExtensions::FromSnakeCase(string)

for converting strings from snake_case to NotSnakeCase

@NaoUnderscore NaoUnderscore added enhancement New feature or request requires-testing Things need to be verified by an Exiled Developer/Contributor P2 Medium priority labels Jan 14, 2024
@NaoUnderscore NaoUnderscore added this to the Exiled 9 milestone Jan 14, 2024
@VALERA771
Copy link
Contributor Author

Install command tested. Other will be tomorrow

@VALERA771 VALERA771 closed this Jan 17, 2024
@VALERA771 VALERA771 deleted the plugin-installer branch January 17, 2024 18:52
@VALERA771 VALERA771 restored the plugin-installer branch January 17, 2024 18:52
@VALERA771 VALERA771 reopened this Jan 17, 2024
@VALERA771
Copy link
Contributor Author

All commands have been tested

@VALERA771 VALERA771 closed this Apr 29, 2024
@VALERA771 VALERA771 deleted the plugin-installer branch April 29, 2024 14:02
@VALERA771 VALERA771 restored the plugin-installer branch April 29, 2024 14:02
@VALERA771 VALERA771 reopened this Apr 29, 2024
@VladTheCow VladTheCow added tested This PR has been built and tested and does not cause any obvious errors. and removed requires-testing Things need to be verified by an Exiled Developer/Contributor labels May 18, 2024
@NaoUnderscore NaoUnderscore added the Do not merge Don't merge this PR label May 21, 2024
@VladTheCow
Copy link
Contributor

Why DoNotMerge?

@Misfiy
Copy link
Contributor

Misfiy commented May 26, 2024

@NaoUnderscore (as asked by vlad) why do not merge?

@NaoUnderscore
Copy link
Collaborator

Because I'd like to wait before merging Plugin Manager/Installer-like features.
We're going to implement it through our Web APIs.

@VALERA771
Copy link
Contributor Author

@NaoUnderscore

@NaoUnderscore NaoUnderscore merged commit 7ebbd79 into Exiled-Team:apis-rework Jun 23, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@VladTheCow VladTheCow VladTheCow approved these changes

@louis1706 louis1706 louis1706 approved these changes

@Misfiy Misfiy Misfiy approved these changes

@NaoUnderscore NaoUnderscore Awaiting requested review from NaoUnderscore

Assignees

@VALERA771 VALERA771

Labels
Do not merge Don't merge this PR enhancement New feature or request P2 Medium priority regarding-api tested This PR has been built and tested and does not cause any obvious errors.
Projects
None yet
Milestone
Exiled 9
Development

Successfully merging this pull request may close these issues.
5 participants
@VALERA771 @VladTheCow @Misfiy @NaoUnderscore @louis1706