Skip to content

fixed landfill deletion#1559

Merged
grilledham merged 2 commits intoRefactorio:developfrom
actinoide:patch-3
Apr 15, 2026
Merged

fixed landfill deletion#1559
grilledham merged 2 commits intoRefactorio:developfrom
actinoide:patch-3

Conversation

@actinoide
Copy link
Copy Markdown
Contributor

fixed landfill deletion upon mining of covering tile.

(sorry for the repeated PRs with tiny fixes, let me know if there is a more elegant way to do this in case i find any more)

fixed landfill deletion upon mining of covering tile.
Copy link
Copy Markdown
Collaborator

@grilledham grilledham left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, I have a comment though. Also have you tested this?

(sorry for the repeated PRs with tiny fixes, let me know if there is a more elegant way to do this in case i find any more)

I think what you are doing is correct. Generally if the issues are unrelated it's favourable to have separate PRs, even if they are small. I personally don't mind small or large PRs, but in my experience some people tend to be more against larger PRs than smaller ones.

Comment thread map_gen/maps/diggy/feature/diggy_hole.lua Outdated
Co-authored-by: grilledham <jrdgillham@gmail.com>
@grilledham grilledham merged commit ca5e720 into Refactorio:develop Apr 15, 2026
1 check passed
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.

2 participants