p4lang / p4c

P4_16 reference compiler
https://p4.org/
Apache License 2.0
683 stars 446 forks source link

Fix: broken output in Documentation Code Snippets #4919

Open AdarshRawat1 opened 2 months ago

AdarshRawat1 commented 2 months ago

Recent Update to PR

I'm pushing a temporary fix for the code output renders. I'll be reaching out to the Doxygen team about the issue, and hopefully, it will be resolved in the next release. The issue arises when we include one README file within another; in this case, the code blocks are not processed correctly.

What was done

I made a copy of broken code snippets

Changes

Minor fixes in documentation output.

Screenshots of Changes - Current - https://p4lang.github.io/p4c/intermediate_representation_ir.html#visitors-and-transforms ![image](https://github.com/user-attachments/assets/cb0b6197-8468-4f87-bdd0-0fdfdd0a21a9) - Updated ![image](https://github.com/user-attachments/assets/a1a30c97-3cae-49c1-8809-34636cc3e961)
Screenshots of Broken codes for multiple Markdowns - Broken Code snippets for merged Markdown files ![image](https://github.com/user-attachments/assets/c52fe71d-7102-4f1b-b0be-b79395c0dedd) - Fixed ![image](https://github.com/user-attachments/assets/941ea7ba-a5bf-402b-ae6e-5394292a98c7)
github-actions[bot] commented 2 months ago
githubloading A preview of this PR is available at: Preview Link
📂 View the source code here: View Source Code
🔧 Commit used for deployment: dd0dd422f823163adb5ad4a2f76eee4745375586

Note: Changes may take a few seconds to appear on GitHub Pages. Please refresh the page if you do not see the updates immediately.

fruffy commented 2 months ago

Is this ready for review?

AdarshRawat1 commented 1 month ago

Is this ready for review?

I was a bit occupied with university formalities, I'll complete this ASAP.

AdarshRawat1 commented 2 weeks ago

@fruffy Ready for merge !