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

Move chemical plant to long-term life support, and some other minor stuff #2483

Closed
wants to merge 18 commits into from

Conversation

Clayell
Copy link
Contributor

@Clayell Clayell commented Jan 16, 2025

The chemical plant has no processes available until this tech anyway, so having it present decades early only serves to confuse people with the similar-looking ECLSS.

Reported by ezsnack (https://discord.com/channels/319857228905447436/620690446540341261/1329520956628140103)

@Clayell
Copy link
Contributor Author

Clayell commented Feb 3, 2025

also fix some typos in comments (ignore the tech tree commits, that was the wrong branch)

@Clayell
Copy link
Contributor Author

Clayell commented Feb 4, 2025

Copy KSP-RO/ROSolar#22 as well

@Clayell Clayell changed the title Move chemical plant to long-term life support Move chemical plant to long-term life support, and some other minor stuff Feb 4, 2025
hm, how did i miss this in 2365?
Clayell added a commit to Clayell/RP-1 that referenced this pull request Feb 6, 2025
(should be done with KSP-RO#2483)
Clayell added a commit to Clayell/RP-1 that referenced this pull request Feb 6, 2025
(should be done with KSP-RO#2483)
Clayell added a commit to Clayell/RP-1 that referenced this pull request Feb 6, 2025
(should be done with KSP-RO#2483)
Clayell added a commit to Clayell/RP-1 that referenced this pull request Feb 6, 2025
(should be done with KSP-RO#2483)
Clayell added a commit to Clayell/RP-1 that referenced this pull request Feb 6, 2025
(should be done with KSP-RO#2483)
Clayell added a commit to Clayell/RP-1 that referenced this pull request Feb 6, 2025
(should be done with KSP-RO#2483)
Clayell added a commit to Clayell/RP-1 that referenced this pull request Feb 6, 2025
(should be done with KSP-RO#2483)
@Capkirk123
Copy link
Member

There's a lot of changes here that look like they shouldn't be here

@Clayell
Copy link
Contributor Author

Clayell commented Feb 15, 2025

These are some generic typos and such, are any of these commits problematic to you? I didn't want to make 10 PR's for minor changes.

@Capkirk123
Copy link
Member

I'm not merging this out of principle

@Clayell
Copy link
Contributor Author

Clayell commented Feb 17, 2025

Ok, fine.

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