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

Clean up some deprecation usages #3698

Merged
merged 46 commits into from
Jan 4, 2025
Merged

Clean up some deprecation usages #3698

merged 46 commits into from
Jan 4, 2025

Conversation

serenibyss
Copy link
Member

@serenibyss serenibyss commented Dec 23, 2024

Removes a lot of usage/overrides of deprecated methods, primarily checkMachine(ItemStack), stopMachine(), and criticalStopMachine()

Commit descriptions go into more detail for each machine, though it should be relatively straightforward

- Remove usages of deprecated stopMachine()
- Switch to checkProcessing() from checkMachine()
- Delete unused method onRunningTickTabbedMode()
- Move away from deprecated stopMachine() override
- Remove deprecated onWireCutterRightClick() usage
- Cleanup spawnVisualPistonBlocks() and getAllBlockTiers() warnings
- Remove deprecated stopMachine() usage
- Remove unnecessary cast
- Remove 2 identical overrides
- remove usage of deprecated stopMachine()
- remove usage and override of deprecated stopMachine()
- Remove two unused methods
- Remove one identical override
- Use checkProcessing() over checkRecipe()
- Fix cast warning
- Move to checkProcessing() over checkRecipe()
- Move away from deprecated stopMachine()
- Clean up try-catch methods
- move to checkProcessing() over checkRecipe()
- remove 1 method identical to super
- Use checkProcessing() over checkRecipe()
- Remove 1 method identical to super
- Use checkProcessing() over checkRecipe()
- Remove usage of deprecated stopMachine() and criticalStopMachine()
- Use checkProcessing() over deprecated checkRecipe()
- Remove override identical to super
- Two warnings for lambda usage
@serenibyss serenibyss requested review from mitchej123 and a team and removed request for mitchej123 December 23, 2024 17:23
@serenibyss serenibyss added the refactor For PRs rewritting a part of the code to have a nicer code overall. label Dec 23, 2024
serenibyss and others added 15 commits December 23, 2024 11:24
- Use checkProcessing() over checkRecipe()
- Remove deprecated stopMachine() override
- Use checkProcessing() over checkRecipe()
- Move away from deprecated stopMachine()
- Use checkProcessing() over checkRecipe()
- Remove deprecated stopMachine() override
- Use checkProcessing() over checkRecipe()
Use checkProcessing() over checkRecipe()
Rename some variables for readability
- remove usage of deprecated stopMachine()
- move away from deprecated onWireCutterRightClick()
- move away from deprecated onSolderingToolRightClick()
- move away from deprecated onWireCutterRightClick() and onSolderingToolRightClick()
- move away from deprecated onSolderingToolRightClick()
- move away from deprecated onWrenchRightClick()
- move away from deprecated onWireCutterRightClick()
- move away from deprecated onWireCutterRightClick()
- move away from deprecated onWireCutterRightClick()
serenibyss and others added 12 commits December 23, 2024 19:44
- move away from deprecated onWireCutterRightClick()
- remove 3 methods identical to super
- remove unused addToFRFList()
- move away from deprecated onWireCutterRightClick()
- move away from deprecated onWireCutterRightClick()
- move away from deprecated onWireCutterRightClick()
- move away from deprecated onWireCutterRightClick()
- move away from deprecated onWireCutterRightClick()
- move away from deprecated survivalConstruct()
- move away from deprecated onWireCutterRightClick()
- move away from deprecated survivalConstruct()
- move away from deprecated onWireCutterRightClick()
- move away from deprecated survivalConstruct()
- move away from deprecated onWireCutterRightClick()
- move away from deprecated survivalConstruct()
- move away from deprecated stopMachine()
- remove unused addEnergyHatchToMachineList()
- remove 4 methods identical to super
- clean up smaller warnings
@Dream-Master Dream-Master requested a review from a team December 26, 2024 02:02
@serenibyss serenibyss added the 🚧 Testing on Zeta Do not merge yet, testing this PR on Zeta label Dec 29, 2024
@Dream-Master Dream-Master enabled auto-merge (squash) January 4, 2025 15:46
@Dream-Master Dream-Master merged commit 8107f16 into master Jan 4, 2025
5 checks passed
@Dream-Master Dream-Master deleted the deprecations-cleanup branch January 4, 2025 15:53
@Dream-Master Dream-Master removed the 🚧 Testing on Zeta Do not merge yet, testing this PR on Zeta label Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor For PRs rewritting a part of the code to have a nicer code overall.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants