Skip to content

Conversation

dt
Copy link
Member

@dt dt commented Jun 14, 2016

This also moves the headers out of the generated includes to the containing
template -- this should make it a little easier to include hand-written
prose introducing each section (e.g. how aggregates behave).


This change is Reviewable

This also moves the headers out of the generated includes to the containing
template -- this should make it a little easier to include hand-written
prose introducing each section (e.g. how aggregates behave).
@dt dt assigned knz Jun 14, 2016
@knz
Copy link
Contributor

knz commented Jun 14, 2016

Awesome, thanks!

:lgtm_strong:


Reviewed 5 of 5 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@jseldess
Copy link
Contributor

:lgtm:


Reviewed 5 of 5 files at r1.
Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.


Comments from Reviewable

@jseldess jseldess merged commit faab4ce into cockroachdb:gh-pages Jun 14, 2016
@dt dt deleted the aggs branch June 14, 2016 18:15
Simran-B pushed a commit to Simran-B/docs that referenced this pull request Aug 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants