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

WIP: docs: add a script that generates FEATURES.md #1371

Draft
wants to merge 5 commits into
base: dev
Choose a base branch
from

Conversation

alufers
Copy link
Contributor

@alufers alufers commented Nov 10, 2022

This script takes FEATURES.src.md and finds all ```pwndbg blocks.
Then it executes them with pwngdb and creates an svg screenshot.
It can also compile some basic C programs and adds some annotations to the screenshots.

You can see the generated file here: https://github.com/alufers/pwndbg/blob/docs/new-features-docs/docs/FEATURES.md

I will add some more docs with examples to this file.

This script takes FEATURES.src.md and finds all ```pwndbg blocks.
Then it executes them with pwngdb  and creates an svg screenshot.
It can also compile some basic C programs and adds some annotations to the screenshots.
@codecov-commenter
Copy link

Codecov Report

Merging #1371 (30cf09b) into dev (6f2f527) will not change coverage.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##              dev    #1371   +/-   ##
=======================================
  Coverage   59.26%   59.26%           
=======================================
  Files         174      174           
  Lines       20971    20971           
  Branches     1848     1848           
=======================================
  Hits        12429    12429           
  Misses       7945     7945           
  Partials      597      597           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@gsingh93
Copy link
Member

Ping 🙂

If there's more work to do here, we can always get what's currently implemented shipped and then come back to the remaining work later.

@gsingh93
Copy link
Member

Ping

@alufers
Copy link
Contributor Author

alufers commented Jun 20, 2023

Hi, haven't had much time lately, but I still remember about this. If you fell like it clutters up the pull requests you can close it. I will send another one when it's ready.

@gsingh93
Copy link
Member

Hi, haven't had much time lately, but I still remember about this. If you fell like it clutters up the pull requests you can close it. I will send another one when it's ready.

No worries, can you mark it as a draft PR for now? And again, whenever you get time to work on this we can merge any smaller bits of it that are done even if the whole thing isn't complete.

@alufers alufers marked this pull request as draft June 20, 2023 17:52
@gsingh93 gsingh93 added this to the 2023.10 milestone Jul 17, 2023
@gsingh93 gsingh93 modified the milestones: 2024.01, Future Apr 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants