Closed isaacsas closed 6 months ago
@gzagatti I am ok to merge this if my updates to the simple TPP tutorial are ok with you. Please see my one question.
I haven't had time to carefully go through your more involved application that adds an interface to PointProcesses, but am ok with merging now that we've moved it to an application and I clarified the language that isn't a basic tutorial.
Totals | |
---|---|
Change from base Build 7191453830: | 0.0% |
Covered Lines: | 2299 |
Relevant Lines: | 2556 |
@isaacsas it looks all good to me. I have corrected a few typos and fixed a dangling reference. I can't push to this branch, but you can find my changes here. I guess you can just merge.
I'm good to have the PR merged after these fixes.
Ok, push the version with your additions as a v5 PR and I can merge it after tests pass tomorrow morning.
@gzagatti I made (am making) some more updates. I can't push to your branch either as I think you have this disabled on your fork.
Let's just work in this version. If you can't push to it let me know and I'll try to get that straigtened out.