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

Add supports to deferred #10346

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Add supports to deferred #10346

wants to merge 2 commits into from

Conversation

Clayell
Copy link

@Clayell Clayell commented Dec 28, 2024

Add supports from https://github.com/LGhassen/Deferred?tab=readme-ov-file#mod-compatibility-status.
This PR is currently a draft, as I am waiting for a response relating to an incompatibility with Procedural Fairings. (KSP-RO/ProceduralFairings#59)

There are certain mods that seem to have been always compatible with the public versions of deferred or where deferred itself was seemingly updated to be compatible with these mods, as the mods themselves do not have any patch notes relating to deferred compatibility.
Additionally, if a mod seems to have had multiple patch notes relating to deferred, I went with the most recent one as the supported one.
I have left out Planetshine, as even though it is technically compatible, as LGhassen states it does not a reason to be used with deferred under normal settings, which is what a majority of users will be using.
I also left out the ASET props mod(s) as its compatibility depends on RPM as stated by the mod author.

@SofieBrink
Copy link
Contributor

Could add tundra if you’d like
https://github.com/TundraMods/TundraExploration/releases/tag/7.1.1

@Clayell
Copy link
Author

Clayell commented Dec 28, 2024

Could add tundra if you’d like

https://github.com/TundraMods/TundraExploration/releases/tag/7.1.1

Considering that the compatibility isn't mentioned in the Deferred readme, it would likely be better to mark Deferred as compatible with Tundra in the Tundra netkan, rather than the other way around.

@SofieBrink
Copy link
Contributor

Yeah the relationship exists the other way already. Fair enough

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