Merge Trails and Geomatics

[HakanSeven12]

This PR created to merge geomatics wb and trails wb

[lgtm-com[bot]]

This pull request introduces 291 alerts when merging ea233e7f505b4f7c837f0a6d695c3a723228f629 into 3b90bca28a040d75094614808eb12febb55cc7ae - view on LGTM.com

new alerts:

• 86 for Except block handles 'BaseException'
• 51 for Unused local variable
• 42 for Variable defined multiple times
• 37 for Unused import
• 29 for Unreachable code
• 16 for Testing equality to None
• 16 for 'import *' may pollute namespace
• 5 for Constant in conditional expression or statement
• 3 for Unnecessary pass
• 2 for Use of 'global' at module level
• 1 for First parameter of a method is not named 'self'
• 1 for Use of the return value of a procedure
• 1 for Redundant assignment
• 1 for Nested loops with same variable

[lgtm-com[bot]]

This pull request introduces 135 alerts when merging 568128f754e4bb08624683f03400ee872d017431 into 3b90bca28a040d75094614808eb12febb55cc7ae - view on LGTM.com

new alerts:

• 81 for Except block handles 'BaseException'
• 19 for Unused import
• 16 for 'import *' may pollute namespace
• 9 for Unused local variable
• 5 for Variable defined multiple times
• 2 for Testing equality to None
• 2 for Unreachable code
• 1 for Nested loops with same variable

[lgtm-com[bot]]

This pull request introduces 110 alerts when merging 31cfc2d362696445ea9da2cc48b6380e8dde706e into 3b90bca28a040d75094614808eb12febb55cc7ae - view on LGTM.com

new alerts:

• 81 for Except block handles 'BaseException'
• 16 for 'import *' may pollute namespace
• 6 for Unused import
• 5 for Variable defined multiple times
• 2 for Unused local variable

[lgtm-com[bot]]

This pull request introduces 27 alerts when merging d841c19aa8bc44175e390fbf82058ba639b76e2a into 3b90bca28a040d75094614808eb12febb55cc7ae - view on LGTM.com

new alerts:

• 16 for 'import *' may pollute namespace
• 6 for Unused import
• 3 for Variable defined multiple times
• 2 for Unused local variable

[lgtm-com[bot]]

This pull request introduces 31 alerts when merging 4a9bf5644f9ca754c0b4c0545fb203e8130c02a6 into 3b90bca28a040d75094614808eb12febb55cc7ae - view on LGTM.com

new alerts:

• 16 for 'import *' may pollute namespace
• 10 for Unused import
• 3 for Variable defined multiple times
• 2 for Unused local variable

[HakanSeven12]

Finished.

[joelgraff]

Did you change your mind about the merge?

[HakanSeven12]

No. I still want to merge.

[joelgraff]

Ok. Well, when you're ready, reopen the request and I'll go over it.

[HakanSeven12]

Okay. Reopened.

[HakanSeven12]

I think it's ready.

[lgtm-com[bot]]

This pull request introduces 16 alerts when merging f0de3fc48d4368a95c454edf33bad9b7fa5d3b90 into 3b90bca28a040d75094614808eb12febb55cc7ae - view on LGTM.com

new alerts:

• 9 for Unused import
• 3 for Variable defined multiple times
• 2 for Unused local variable
• 2 for 'import *' may pollute namespace

[lgtm-com[bot]]

This pull request introduces 16 alerts when merging 39ef55869aa027fd5b5f27f65371e74d899cc9d7 into 3b90bca28a040d75094614808eb12febb55cc7ae - view on LGTM.com

new alerts:

• 9 for Unused import
• 3 for Variable defined multiple times
• 2 for Unused local variable
• 2 for 'import *' may pollute namespace

[lgtm-com[bot]]

This pull request introduces 16 alerts when merging cee89e023d8c34806d4e2ca42537d94a70affde7 into 3b90bca28a040d75094614808eb12febb55cc7ae - view on LGTM.com

new alerts:

• 9 for Unused import
• 3 for Variable defined multiple times
• 2 for Unused local variable
• 2 for 'import *' may pollute namespace

[joelgraff]

Thanks for counting to push changes. I’ll look at merging when I can, but things have been busy. Maybe in the next week or so...

On Fri, Mar 27, 2020 at 6:54 AM lgtm-com[bot] notifications@github.com wrote:

This pull request introduces 16 alerts when merging cee89e0 https://github.com/joelgraff/freecad.trails/commit/cee89e023d8c34806d4e2ca42537d94a70affde7 into 3b90bca https://github.com/joelgraff/freecad.trails/commit/3b90bca28a040d75094614808eb12febb55cc7ae

• view on LGTM.com https://lgtm.com/projects/g/joelgraff/freecad.trails/rev/pr-0db3f65ff535fcbfd056631735e997d370a13cdc

new alerts:

• 9 for Unused import
• 3 for Variable defined multiple times
• 2 for Unused local variable
• 2 for 'import *' may pollute namespace

— You are receiving this because you commented.

Reply to this email directly, view it on GitHub https://github.com/joelgraff/freecad.trails/pull/55#issuecomment-604959748, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA2VMSTUDROMG23WN2YZDZTRJSHZHANCNFSM4LQ3D6PQ .

-- Joel Graff, P.E.

LinkedIn: https://www.linkedin.com/in/joelcgraff

[HakanSeven12]

You are welcome :)

[lgtm-com[bot]]

This pull request introduces 15 alerts when merging 6995895aacc3889f0451c580da05f0769a730d15 into 3b90bca28a040d75094614808eb12febb55cc7ae - view on LGTM.com

new alerts:

• 8 for Unused import
• 3 for Variable defined multiple times
• 2 for Unused local variable
• 2 for 'import *' may pollute namespace

[lgtm-com[bot]]

This pull request introduces 16 alerts when merging 13b9de6450db1ad4b98ae52f5411c249cc29436e into 3b90bca28a040d75094614808eb12febb55cc7ae - view on LGTM.com

new alerts:

• 8 for Unused import
• 3 for Unused local variable
• 3 for Variable defined multiple times
• 2 for 'import *' may pollute namespace

[lgtm-com[bot]]

This pull request introduces 17 alerts when merging 6b912448686d3ef6e379a9938666303fd854de50 into 3b90bca28a040d75094614808eb12febb55cc7ae - view on LGTM.com

new alerts:

• 8 for Unused import
• 4 for Unused local variable
• 3 for Variable defined multiple times
• 2 for 'import *' may pollute namespace

[lgtm-com[bot]]

This pull request introduces 17 alerts when merging f69f22696c45560206e27a175dbbf72352541b94 into 3b90bca28a040d75094614808eb12febb55cc7ae - view on LGTM.com

new alerts:

• 8 for Unused import
• 4 for Unused local variable
• 3 for Variable defined multiple times
• 2 for 'import *' may pollute namespace