-
Notifications
You must be signed in to change notification settings - Fork 8
Add RTS options reading from argv #57
Conversation
Oh, I also added |
@garyb Looks good! Thanks for the update. Should I merge and make a release? |
Yes please! |
Just noticed, I think we need the Also, should we add any documentation about the RTS option? I was looking around and found some info on it here: However, do you think there is something else we should use. Or maybe we don't need specific documentation about it. What do you think? |
Good catch 😄 Yeah, I'm not sure how much documentation it needs - if someone has a need to use it they're probably already familiar with the RTS options. I generally have to do a lot of looking up whenever I need to figure out how to use them too, and I've only done basic things like setting the concurrency, memory limit, and debug tracing. You're right that we should mention the fact it exists at all though. |
Ok, I've updated the fold and the docs to include the RTS info. We also had a mistake in the existing docs where |
Great! Thanks for the update and the documentation fix. |
Add RTS options reading from argv
Thanks for the review and release! |
Welcome, thanks for the PR |
@ethul how does this look?