Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: fix multiline comments from proto files breaking tables #1431

Merged
merged 3 commits into from
Sep 22, 2023

Conversation

constantoine
Copy link
Contributor

FIX: This patch changes how documentation generated from proto files is rendered to fix multiline comments breaking tables.

Related issue(s)

ory/docs#1537

Checklist

  • I have read the contributing guidelines.
  • I have referenced an issue containing the design document if my change
    introduces a new feature.
  • I am following the
    contributing code guidelines.
  • I have read the security policy.
  • I confirm that this pull request does not address a security
    vulnerability. If this pull request addresses a security vulnerability, I
    confirm that I got the approval (please contact
    [email protected]) from the maintainers to push
    the changes.
  • I have added tests that prove my fix is effective or that my feature
    works.
  • I have added or changed the documentation.

Further Comments

The project github.com/pseudomuto/protoc-gen-doc seems to be unmaintained, so the fix for this bug never made it to upstream.

@CLAassistant
Copy link

CLAassistant commented Sep 18, 2023

CLA assistant check
All committers have signed the CLA.

@constantoine constantoine changed the title Fix multiline comments from proto files fix: multiline comments from proto files breaking tables Sep 18, 2023
Signed-off-by: Cléo Rebert <[email protected]>
Copy link
Member

@zepatrik zepatrik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome!

@@ -0,0 +1,107 @@
<!-- This file is automatically generated. DO NOT EDIT! -->
<!-- Needed because of https://github.com/pseudomuto/protoc-gen-doc/issues/274 -->
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added a reference to the issue, because then our github action will notify us when it is fixed upstream 😉

@zepatrik zepatrik changed the title fix: multiline comments from proto files breaking tables docs: fix multiline comments from proto files breaking tables Sep 22, 2023
@zepatrik zepatrik enabled auto-merge September 22, 2023 12:29
@zepatrik zepatrik disabled auto-merge September 22, 2023 12:29
@zepatrik zepatrik enabled auto-merge September 22, 2023 12:30
@zepatrik zepatrik added this pull request to the merge queue Sep 22, 2023
Merged via the queue into ory:master with commit ef9132d Sep 22, 2023
21 checks passed
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