Skip to content
This repository was archived by the owner on Dec 30, 2023. It is now read-only.

Remove plugin solution, add string names generator#49

Merged
Richasy merged 1 commit intomainfrom
richasy/splitPlugin
Jun 23, 2023
Merged

Remove plugin solution, add string names generator#49
Richasy merged 1 commit intomainfrom
richasy/splitPlugin

Conversation

@Richasy
Copy link
Owner

@Richasy Richasy commented Jun 23, 2023

Description

This is a code refactoring endeavor.

  1. Migrate the CorePlugin project to a new repository.
  2. Utilize the Source Code Generator to automatically generate enumerations when modifying text resource files.

PR type

What is the purpose of this PR?

  • Code style

What is the current behavior?

  1. Modifications to the core repository may introduce pull requests that are unrelated to the main project, which, in the long run, can impede maintainability.
  2. Every time I add new textual resources, I always have to begin by incorporating new enumerations, followed by modifying the resw file. It can be quite cumbersome at times.

What is the new behavior?

  1. Divide the project in order to separately maintain the core library.
  2. Introducing a source code generator would allow for the modification of the resw files, thereby enabling the automatic generation of enumerations.

PR checklist

Please check that your PR meets the following requirements:

  • App successfully launched
  • File headers have been added to all source files
  • NOT Contains breaking updates

@Richasy Richasy added this to the 2306 Release milestone Jun 23, 2023
@Richasy Richasy merged commit 43527c2 into main Jun 23, 2023
@Richasy Richasy deleted the richasy/splitPlugin branch June 23, 2023 06:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant