Skip to content

add more delete comments for IGenericModConfigMenuApi.cs - #513

Closed
ifenghm wants to merge 1 commit into
spacechase0:developfrom
ifenghm:develop
Closed

ifenghm wants to merge 1 commit into
spacechase0:developfrom
ifenghm:develop

Conversation

@ifenghm

@ifenghm ifenghm commented May 27, 2025

Copy link
Copy Markdown
Contributor

I noticed when making a custom ModConfig and importing this code, we should also remove the UsingGenericModConfigMenu.Framework code (since that is not actually imported in our source code)

I noticed when making a custom ModConfig and importing this code, we should also remove the UsingGenericModConfigMenu.Framework code (since that is not actually imported in our source code)
@spacechase0

Copy link
Copy Markdown
Owner

So apparently my reorganizing the repository created conflicts for every single PR. I figured the PR system would be smart enough to handle files being moved around fine, but I guess not.

I plan on manually merging this soon with some rebase shenanigans, so no need to worry about fixing this. I'll leave the PR open until it's taken care.

@spacechase0

Copy link
Copy Markdown
Owner

Manually merged.

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