envoyproxy / envoyproxy/envoy

Move protodoc to `envoy_py_binary` and use a single jinja template to render

Open
#16,328 2 comments 0 reactions 1 assignee Claimed by @phlax View on GitHub
area/docs area/protobuf area/python code help wanted
Dominant language
C++
Stars
28.9k
Forks
5.6k
Avg merge
1d 20h
Merged PRs (30d)
437

Description

### description

protodoc will benefit quite a lot from moving to the newer `envoy_py_binary` format and moving tests to pytest

what would also greatly simplify the code is using the existing visitor pattern to collect up a py dictionary of the page structure and injecting that to a single template, rather than lots of snippets and `\n` workarounds

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.