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

Fix Doc typos in ThreatModel & Added README on how to build docs #4058

Closed
wants to merge 5 commits into from

Conversation

kskarthik
Copy link
Contributor

@kskarthik kskarthik commented Apr 22, 2024

Added build instructions for the docs & a README file in the /documentation section pointing it.

Before you submit a pull request, please make sure:

  • The pull request should include a description of the problem you're trying to solve
  • The pull request should include overview of the suggested solution
  • If the pull requests changes current behavior, reasons why your solution is better.
  • The proposed code should be fully functional
  • The proposed code should contain tests relevant to prove is functionality
  • The proposed tests should ensure significative code coverage
  • All new and existing tests should pass

@evilaliv3
Copy link
Member

Thank you @kskarthik !

I've manually integrated some of your fixes and assigned the commit to you to maintain your credit.

9a405c0

@evilaliv3 evilaliv3 closed this Aug 25, 2024
@evilaliv3
Copy link
Member

Same made for:

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.

2 participants