Skip to content

add mixing of required mods and freemod to !mp mods#89

Merged
ILW8 merged 3 commits intomasterfrom
feature/fm-with-req-mods
Sep 25, 2025
Merged

add mixing of required mods and freemod to !mp mods#89
ILW8 merged 3 commits intomasterfrom
feature/fm-with-req-mods

Conversation

@ILW8
Copy link
Copy Markdown
Owner

@ILW8 ILW8 commented Sep 24, 2025

Example: !mp mods FM+NF forces NF and allows freemod (while removing SD from freemod as it isn't compatible with NF).

@ILW8 ILW8 requested a review from Copilot September 25, 2025 00:02
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enhances the !mp mods command to support mixing required mods with freemod functionality. Previously, the command only supported either freemod (FM) or regular mods, but now allows combinations like FM+NF to force specific mods while enabling freemod for others.

  • Refactored mod parsing logic to process each mod individually instead of treating freemod as a special case
  • Added compatibility checking to remove incompatible mods from the allowed freemod list when required mods are present
  • Improved mod conflict resolution by removing duplicate required mods from allowed mods list

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

ILW8 and others added 2 commits September 25, 2025 01:06
@ILW8 ILW8 merged commit 2c81039 into master Sep 25, 2025
3 checks 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