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

Remove references deprecated in MCprep 3.5 #521

Merged
merged 2 commits into from Dec 28, 2023

Conversation

StandingPadAnimations
Copy link
Collaborator

This PR removes the deprecated conf.py references that were meant to be removed in MCprep 3.5.1, but weren't due to Blender 4.0's release.

Most of these are related to the transition from global variables in conf.py to a proper global state with MCprepEnv and the env variable, but some also deal with ending 2.7X support, such as make_annotations or bv28 (although bv28's removal could be delayed if we haven't removed all uses of it yet)

This commit removes the deprecated conf.py references that were meant to
be removed in MCprep 3.5.1, but weren't due to Blender 4.0's release
@StandingPadAnimations StandingPadAnimations requested a review from a team December 27, 2023 22:53
@TheDuckCow TheDuckCow added this to the v3.5.2 milestone Dec 28, 2023
Copy link
Member

@TheDuckCow TheDuckCow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed all tests still passing, so unlikely any forgotten references

-------------------------------------------------------------------------------
bversion   	ran_tests	ran	skips	failed	errors
-------------------------------------------------------------------------------
(3.6.2)   	all_tests	63	2	0	No errors
(4.0.2)   	all_tests	63	2	0	No errors
(3.5.1)   	all_tests	63	2	0	No errors
(3.4.0)   	all_tests	63	2	0	No errors
(3.3.1)   	all_tests	63	2	0	No errors
(3.2.1)   	all_tests	63	2	0	No errors
(3.1.0)   	all_tests	63	2	0	No errors
(3.0.0)   	all_tests	63	2	0	No errors
(2.93.0)   	all_tests	62	3	0	No errors
(2.90.1)   	all_tests	62	3	0	No errors
(2.80.75)   	all_tests	61	4	0	No errors
tests took 281s to run, ending with code 0

@StandingPadAnimations
Copy link
Collaborator Author

Alright, then I'll merge the changes

@StandingPadAnimations StandingPadAnimations merged commit 5911cea into dev Dec 28, 2023
@StandingPadAnimations StandingPadAnimations deleted the remove_3-5_deprecated-references branch December 28, 2023 22:54
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.

None yet

2 participants