Skip to content

Commit

Permalink
Update tab_contributing.md
Browse files Browse the repository at this point in the history
Added details for contributing to ISTG based on MASTG
  • Loading branch information
scriptingxss authored Dec 14, 2023
1 parent 325c5f3 commit 6adfcd6
Showing 1 changed file with 83 additions and 4 deletions.
87 changes: 83 additions & 4 deletions tab_contributing.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,16 +9,95 @@ tags: istg

## Contributing to the ISTG Project

***Work in Progress:** This page is currently under construction. The information displayed is not yet complete.*
_First of all,_ [โญ Please Give us a Star in GitHub](https://github.com/OWASP/owasp-istg)!

<br>

Thank you for your support and dedication to IoT security. The ISTG project is an open source effort and we welcome all kinds of contributions and feedback. To contribute, please head over to our [GitHub Repository][istg_github]. Here, you can review the projectโ€™s documentation, code, and share your valuable feedback. Your expertise and insights will play a crucial role in improving the guideโ€™s quality and relevance. Whether you are an experienced IoT security tester or someone passionate about ensuring the security of connected devices, your contributions are highly welcome. Join us in this collaborative effort to strengthen IoT security testing practices and make a positive impact on the industry!

To contribute, please head over to our [GitHub Repository][istg_github]. Here, you can review the projectโ€™s documentation, code, and share your valuable feedback. Your expertise and insights will play a crucial role in improving the guideโ€™s quality and relevance.
## ๐Ÿ’ฌ Participate in Discussions

Whether you are an experienced IoT security tester or someone passionate about ensuring the security of connected devices, your contributions are highly welcome. Join us in this collaborative effort to strengthen IoT security testing practices and make a positive impact on the industry!
Our [GitHub Discussions](https://github.com/OWASP/owasp-istg/discussions) are the first place to go to ask questions, give feedback, and propose new ideas. If your proposal qualifies for the ISTG, we'll convert it into an "Issue" (the discussion might take a while).

Thank you for your support and dedication to IoT security. Together, we can make a difference.
**Help us improve & join our community:**

- ๐Ÿž Report an error (typos, grammar) or fix it on a Pull Request.
- ๐Ÿ’ฌ [Give feedback](https://github.com/OWASP/owasp-istg/discussions/categories/general).
- ๐Ÿ™ [Ask Questions](https://github.com/OWASP/owasp-istg/discussions/categories/q-a).

## ๐ŸŽฏ Create Issues

Before creating a PR, first create an [Issue](https://github.com/OWASP/owasp-istg/issues "ISTG Issues") to be discussed for missing requirements, content or errors.

- To avoid multiple people duplicating effort on the same issue, project leaders will assign it to only a few that will own it.
- Explain what you think is missing in the issue, including references (if available) and suggest where it could be added.

## ๐Ÿ“ Open a Pull Request

You can contribute with content or corrections by opening a Pull Request (PR). You can create a PR by following [these steps](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request-from-a-fork).
Remember that:
- The target branch should be `master`.
- If your PR closes and issue, write ["Closes `#<issue-id>`"](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue).

Your PR will be reviewed soon (refer to this page to learn more about [reviews](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/about-pull-request-reviews)).


- Your PR may be merged after review.

> Learn more: ["(GitHub Docs) Reviewing proposed changes in a pull request"](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/reviewing-proposed-changes-in-a-pull-request).
## ๐Ÿ“Contribute with content:

- ๐Ÿ’ก [Propose ideas or suggest improvements](https://github.com/OWASP/owasp-istg/discussions/categories/ideas). If it qualifies we'll promote it to an Issue.
- ๐Ÿ“„ Create a Pull Request for concrete fixes (e.g. grammar/typos) or content already approved by the core team.


## ๐Ÿšซ What not to do

Although we greatly appreciate any and all contributions to the project, there are a few things that you should take into consideration:

- **No advertisement**: The OWASP IoT Security Project cannot be used as a platform for advertisement of commercial tools, companies or individuals. Technical content such as the implementation of certain techniques or tests should be written with free and open-source tools in mind. Commercial tools are typically not accepted, but might be referenced in some specific cases.
- **No unnecessary self-promotion of tools or blog posts**: If you have a relation with one of the URLs or tools you are referencing, please state so in the PR so that we can verify that the reference is in line with the rest of the guide.

If you have any doubts, please contact us.

<br>

## โœ… Become a Reviewer

You can [Review Pull Requests (PRs)](https://github.com/OWASP/owasp-istg/pulls) and also gain contributions. If you are a fluent speaker in different languages, feel free to give feedback on any of the submitted PRs.

After your PR or issue has been submitted, we will review it as quickly as possible which typically only takes a few days. If you think we have forgotten about it, feel free to give us a nudge after 7 days have passed.

## ๐Ÿ”Ž Proof-reading

If you do proof-reading, these are the things weโ€™re looking for:

- Content [cohesion & coherence](https://writing.chalmers.se/chalmers-writing-guide/writing-a-text/coherence-cohesion/)
- is there a good linkage of ideas?
- does the paragraph make sense?
- does it make sense with the next one? think that hundreds of people have written in here, often without considering the surroundings of the text they were including).
- Reducing the content to a minimum (people tend to be very verbose/wordy) and in such a document we need clear and short/concise statements
- Optimize for _scannability_ (maybe instead of a big paragraph it would be better to have a bullet point list).
- Any passive voice sentences? Convert to active voice.
- Does each paragraph focus on a single topic?
- Are key points stated at the start of each section?
- Are commas, parentheses, colons, em-dashes, and semicolons used properly?

Refer to Google Technical Writing trainings for more info:

- [Google Technical Writing One](https://developers.google.com/tech-writing/one)
- [Google Technical Writing Two](https://developers.google.com/tech-writing/two)

## ๐ŸŒ Translating the ISTG

Translating the ISTG in a new language is another great way to contribute. This helps the project to reach to more people around the world.

Before starting a translation please consider the following:

- **PLEASE** contact us on Slack or via email.
- We need your commitment. After the first translation is done, we will ask for your help to translate any new changes, so your translation can remain up to date.
- We need a second translator who can verify that the English version of the ISTG has been translated properly.


[istg_github]: https://github.com/OWASP/owasp-istg

0 comments on commit 6adfcd6

Please sign in to comment.