Doc fixes (add wedge)

[revolter]

No description provided.

[codecov]

Codecov Report

Merging #844 (631b4ef) into master (cd0c81e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #844   +/-   ##
=======================================
  Coverage   96.11%   96.11%           
=======================================
  Files          39       39           
  Lines        9130     9130           
  Branches     1017     1017           
=======================================
  Hits         8775     8775           
  Misses        206      206           
  Partials      149      149           

Impacted Files	Coverage Δ
cadquery/cq.py	91.76% <ø> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cd0c81e...631b4ef. Read the comment docs.

[jmwright]

@revolter Why did you remove the mention of the wedge primitive?

Wedge:

cadquery/cadquery/cq.py

Line 3803 in 4c45eb2

def wedge(

[revolter]

Because that section has

See 3-d Operations to learn more.

at the end, which links to https://cadquery.readthedocs.io/en/latest/apireference.html#id1, which doesn't mention wedges nor cylinders.

[jmwright]

Ok, so we'll need to add the missing a reference to wedge under 3-d Operations at some point.

[revolter]

Is it a complicated task? Also, what about cylinder?

[jmwright]

Is it a complicated task?

No, you can just add it below Workplane.sphere under the autosummary here.

Also, what about cylinder?

I did a search to confirm and there is no analogous method for Workplane.cylinder.

[revolter]

Like this?

https://cadquery--844.org.readthedocs.build/en/844/primer.html#id1

=>

https://cadquery--844.org.readthedocs.build/en/844/apireference.html#id1

[jmwright]

Like this?

Yes, that looks correct to me.

Looks like we're going to have to correct a mypy issue that's probably caused by the update to OCP 7.5.2. I'll try to get to that today or tomorrow.

[jmwright]

@adam-urbanczyk I think it may be the missing OCP 7.5.2 stubs that are keeping mypy from passing in these tests. Could you confirm when you are available?

[jmwright]

@revolter Assuming that #858 gets merged, that will negate the need to remove the mention of the cylinder primitive in this PR.

[revolter]

Oh, wow, amazing ❤️ We'll wait for it to get merged, if it's ok, and then I'll update the title and content of this one.

[jmwright]

Thanks @revolter !

[revolter]

And thank you @adam-urbanczyk for updating my PR 😃