Page 1 of 1

Turning Addon PathOpGui.py

Posted: Thu Jul 28, 2022 8:41 am
by WarrenkCash
I edited PathOpGui.py as it was not allowing me to create lath paths with the turning addon. I was asked to create a pull request.

After a couple of hours of reaserch I think the pull request is done. I created a fork of freecad and changed PathOpGui.py .

Origional

Code: Select all

 obj = res.objFactory(res.name, obj=None, parentJob=res.job)
edited

Code: Select all

obj = res.objFactory(res.name, obj=None)


I created a pull request and had an option to merge as it didn't affect anything in the main branch or something like.So I clicked it and got this message on completion.

Pull request successfully merged and closed

You’re all set—the WarrenKCash-patch-1 branch can be safely deleted.

The page is at https://github.com/WarrenKCash/FreeCAD/pull/1

Hopefully this is how it is supposed to work.

Re: Turning Addon PathOpGui.py

Posted: Mon Aug 01, 2022 6:58 pm
by Syres
WarrenkCash wrote: Thu Jul 28, 2022 8:41 am I created a pull request and had an option to merge as it didn't affect anything in the main branch or something like.So I clicked it and got this message on completion.
It appears you created a Pull Request on your own branch not on the FreeCAD master which you cannot Merge because it would need to be reviewed by others first. If you look at https://github.com/FreeCAD/FreeCAD/comp ... sh-patch-1 you'll see a Create Pull Request green button which will take you through to the next stage.

Re: Turning Addon PathOpGui.py

Posted: Mon Aug 01, 2022 7:51 pm
by GeneFC
Syres wrote: Mon Aug 01, 2022 6:58 pm
This has already been sorted out. See

https://forum.freecadweb.org/viewtopic. ... &start=150

Gene