Multi-use CTE's #5485
Multi-use CTE's #5485
Conversation
|
This change is |
There was a problem hiding this comment.
stuff related to multi-use CTEs in the second commit 
Reviewable status:
complete! 1 of 0 LGTMs obtained (waiting on @justinj)
ericharmeling
force-pushed
the
multiple-cte
branch
from
d5c1a0d to
a660cc4
Compare
Yeah that was weird... fixed it. Thanks! |
ericharmeling
requested review from
jseldess and
rmloveland
and removed request for
rmloveland
There was a problem hiding this comment.
Reviewable status:
v19.2/known-limitations.md, line 205 at r1 (raw file):
### Referring to a CTE by name more than once {% include {{ page.version.version }}/known-limitations/cte-by-name.md %}
Can you also delete the include file itself (from _includes/v19.2/known-limitations, since we're no longer using it?
ericharmeling
force-pushed
the
multiple-cte
branch
from
a660cc4 to
f7581ae
Compare
There was a problem hiding this comment.
If you look at the full diff, I updated the examples in common-table-expressions.md to use MovR.
Reviewable status:
v19.2/known-limitations.md, line 205 at r1 (raw file):
Previously, jseldess (Jesse Seldess) wrote…
Can you also delete the include file itself (from
_includes/v19.2/known-limitations, since we're no longer using it?
Done.
- Changed all CTE examples to use MovR - Added multiple-use example - Removed all lifted limitations (from common-table-expressions.md and known-limitations.md)
ericharmeling
force-pushed
the
multiple-cte
branch
from
f7581ae to
88899c6
Compare
There was a problem hiding this comment.
@jseldess I added quick set-up instructions. Should be good to go now.
Reviewable status:
Fixes #5189.
PR includes the following:
common-table-expressions.mdandknown-limitations.md)