Closed KnathanM closed 1 month ago
I've approved the workflows to run - feel free to leave open and work or merge after addressing review comments!
I think best to leave this open until I have worked through the RMG user guide some more in case I see more typos. I was able to install RMG today. I'm also okay to merge this PR whenever though to get it off the list of open PRs.
I think best to leave this open until I have worked through the RMG user guide some more in case I see more typos.
Sounds good to me!
I was able to install RMG today.
Glad to hear it!
I'm also okay to merge this PR whenever though to get it off the list of open PRs.
We have a bot that will mark this as 'stale' if it isn't touched for 90 consecutive days - if that comes to pass, we just merge it then. Otherwise fine to leave it open - we usually have pretty long-term PRs around here.
I propose we merge this in and move any further doc updates into #2704 since we are actively combing through the docs in that PR
sounds good to me
WARNING:root:Initial mole fractions do not sum to one; normalizing. WARNING:root:Initial mole fractions do not sum to one; normalizing. WARNING:root:Initial mole fractions do not sum to one; normalizing. ⚠️ One or more regression tests failed. Please download the failed results and run the tests locally or check the log to see why.
beep boop this comment was written by a bot :robot:
I'm reading through the documentation and noticed some typos in one line. Feel free to merge now if you'd like, or if I see other typos, I'll continue to add them here as a I read.