-
Notifications
You must be signed in to change notification settings - Fork 26
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
"Tweaks" branch not included? #16
Comments
@annabellegoth2boss - I'm not sure what happened here. I thought the changes had been merged, but now I don't think so. Thoughts? |
I have to rush on something else, but I'd like to review & make sure that my changes were either accepted or correctly rejected. I suspect this is easily explained but I haven't had time to check it. |
I bet when we addressed the merge conflicts in the meeting two weeks ago something got missed, or later committed over...taking a look now... |
Yes, they're fine. I'll close this. Jason Perlow (LF editor) developed some proposed editorial fixes. I'll post them as a new branch, but I have to convert them from MS Word first :-(. |
I created a "tweaks" branch with a lot of changes. E.g., it included commit 96d4925 with lots of small changes. Here's the tree.
For example, it says changed "There are two parties involved" to "There are at least two parties involved".
Somehow that didn't get merged in, but I thought I'd sent in a pull request. I'll try again, but this may create some hand-work to work out merge conflicts.
The text was updated successfully, but these errors were encountered: