Skip to content

Comments

Remove some deprecations, mark others for removal#1816

Merged
TechLord22 merged 1 commit intomasterfrom
alson-deprecations
Jun 1, 2023
Merged

Remove some deprecations, mark others for removal#1816
TechLord22 merged 1 commit intomasterfrom
alson-deprecations

Conversation

@ALongStringOfNumbers
Copy link
Contributor

What

This PR removes some deprecated methods/classes that were deemed "old enough" (Done in the 2.5 release cycle or before).
In addition, methods/classes that were marked as deprecated in the 2.6 release cycle were marked as such, with a note about removal in the 2.8 release cycle.

I want to leave at least 1 major version between when something was deprecated and when it is removed, so that there is ample time to addons to adjust to the changes.

There was one deprecation that I did not touch, which is the default enchant in Material. This is now set through the ToolProperty, however I could not figure out a way for the CT method in CTMaterialBuilder to call this new method in the ToolProperty.

Outcome

Removes some deprecations and marks others for eventual removal.

@ALongStringOfNumbers ALongStringOfNumbers added the type: refactor Suggestion to refactor a section of code label Jun 1, 2023
@ALongStringOfNumbers ALongStringOfNumbers requested a review from a team as a code owner June 1, 2023 04:49
@TechLord22 TechLord22 merged commit b34dd59 into master Jun 1, 2023
@TechLord22 TechLord22 deleted the alson-deprecations branch June 1, 2023 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: refactor Suggestion to refactor a section of code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants