Fix "1d" sweep

[MatthiasJ1]

with BuildSketch() as s:
    spine = Polyline((0,0), (1,10), (10,10), mode=PRIVATE)
    sect = spine.wire().perpendicular_line(2, 0)
    sweep(sect, spine, transition=Transition.RIGHT)

Before

After

[codecov]

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (a433707) 96.00% compared to head (f4e3e1a) 95.99%.
Report is 1 commits behind head on dev.

Files	Patch %	Lines
src/build123d/topology.py	90.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##              dev     #484      +/-   ##
==========================================
- Coverage   96.00%   95.99%   -0.02%     
==========================================
  Files          24       24              
  Lines        7458     7461       +3     
==========================================
+ Hits         7160     7162       +2     
- Misses        298      299       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[gumyr]

Fixed with commit 7b5e154. Please follow the contribution steps described here: https://github.com/gumyr/build123d/blob/dev/CONTRIBUTING.md for your next PR, this fix didn't pass code coverage.

[MatthiasJ1]

Your version of the fix passes your code coverage and yet it is less robust. The following is a semantically valid operation which works in my PR but fails in yours:

sweep(Line((0,1), (0,0)) + Line((0,0), (1,-1)), Line(0, 10))

I am happy to make contributions if you find them useful, but I am not willing to comply with arbitrary ancillary requirements. If that is a deal-breaker, I respect your principles, but I'm just doing this for fun.