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.

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

Azurelib Utils Changes V2 #405

Merged
merged 13 commits into from
Aug 16, 2023
Merged

Azurelib Utils Changes V2 #405

merged 13 commits into from
Aug 16, 2023

Conversation

AzureDoom
Copy link
Contributor

@AzureDoom AzureDoom commented Aug 7, 2023

@AzureDoom AzureDoom mentioned this pull request Aug 7, 2023
@JannisX11
Copy link
Owner

I'm currently updating the readme of this repository, so for now reference the Discord post about the new repository format.
Here are the things that need changing:

  • Thze name of the plugin directory should be consistent with the plugin ID.
  • The icon field value should not include the directory, only the file name.
  • The icon should be 48 by 48 pixels.

@AzureDoom
Copy link
Contributor Author

I'm currently updating the readme of this repository, so for now reference the Discord post about the new repository format. Here are the things that need changing:

* Thze name of the plugin directory should be consistent with the plugin ID.

* The icon field value should not include the directory, only the file name.

* The icon should be 48 by 48 pixels.

All three changes have been made now, I have also moved the plugin itself to the directory as well since I see that is possible. Do you need anything to be updated?

@JannisX11
Copy link
Owner

The plugin field value still needs to be updated.

@AzureDoom
Copy link
Contributor Author

The plugin field value still needs to be updated.

Ah sorry, forgot I have to edit that manually for it to preserve the proper formatting, fixed now!

@JannisX11 JannisX11 merged commit 7b7e98e into JannisX11:master Aug 16, 2023
1 check passed
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