Skip to content

chore: cleanup unused action classes - #1213

Merged
Robotgiggle merged 1 commit into
FallingColors:mainfrom
Olfi01:issue/901-remove-unused-actions
Jul 27, 2026
Merged

chore: cleanup unused action classes#1213
Robotgiggle merged 1 commit into
FallingColors:mainfrom
Olfi01:issue/901-remove-unused-actions

Conversation

@Olfi01

@Olfi01 Olfi01 commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Closes #901

All the classes removed are unused in code and have been replaced by Arithmetic implementations.

@Robotgiggle Robotgiggle left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@Robotgiggle
Robotgiggle added this pull request to the merge queue Jul 27, 2026
Merged via the queue into FallingColors:main with commit ec86667 Jul 27, 2026
8 checks passed
@github-project-automation github-project-automation Bot moved this from 📋 Backlog to ✅ Done in Hex Casting Jul 27, 2026
@s5bug s5bug mentioned this pull request Jul 28, 2026
object-Object added a commit that referenced this pull request Jul 29, 2026
Includes PRs:
- #1115 
- #1081 
- #1080 
- #1171 
- #1147 
- #1169
- #1102
- #1204
- #1202
- #1213

Someone more familiar with the datagen than me needs to check why
`scroll_paper.json` isn't being generated under the `Common/` or
`Neoforge/` generated resources. `runAllDatagen` should probably
completely delete the generated resources directory before running just
so stuff like this can be caught at the right time.

---------

Co-authored-by: IridescentVoid <265486018+IridescentVoid@users.noreply.github.com>
Co-authored-by: Robotgiggle <88736742+Robotgiggle@users.noreply.github.com>
Co-authored-by: V. Ntro <251832526+ViktoriaNtro@users.noreply.github.com>
Co-authored-by: ChuijkYahus <94828194+ChuijkYahus@users.noreply.github.com>
Co-authored-by: Chloe <119641630+ChloeTax@users.noreply.github.com>
Co-authored-by: [object Object] <object@objectobject.ca>
Co-authored-by: Florian Meyer <flommeyer@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

Remove unused action classes that are now implemented as arithmetic

2 participants