Skip to content

Commit

Permalink
Merge pull request #956 from floydtree/docu-updates
Browse files Browse the repository at this point in the history
Updating CONTRIBUTION guide, PR Templates
  • Loading branch information
pagbabian-splunk authored Feb 5, 2024
2 parents 26de149 + 1ef74ca commit 596fd07
Show file tree
Hide file tree
Showing 3 changed files with 30 additions and 22 deletions.
2 changes: 2 additions & 0 deletions .github/CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
# Default Owners for everything in the repo
* @floydtree @pagbabian-splunk @Aniak5 @mikeradka @zschmerber
9 changes: 5 additions & 4 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,8 @@
#### Description of changes:

### Delete once you have confirmed the following:
1. Have you followed the [contribution guidelines](https://github.com/ocsf/ocsf-schema/blob/main/CONTRIBUTING.md)?
2. Did you run a local instance of the [ocsf-server](https://github.com/ocsf/ocsf-server) and ensure it ran without any errors/warnings?
3. Have you assigned appropriate labels to the PR?
4. Is your PR title in sync with the description?
1. Did you add a single line summary of changes to `Unreleased` section in the [CHANGELOG.md](https://github.com/ocsf/ocsf-schema/blob/main/CHANGELOG.md) file?
2. Have you followed the [contribution guidelines](https://github.com/ocsf/ocsf-schema/blob/main/CONTRIBUTING.md)?
3. Did you run a local instance of the [ocsf-server](https://github.com/ocsf/ocsf-server) and ensure it ran without any errors/warnings?
4. Have you assigned appropriate labels to the PR?
5. Is your PR title in sync with the description?
41 changes: 23 additions & 18 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,28 @@ This documentation presents guidelines and expected etiquettes to successfully c

* * *

### Key Terminology
### Pull Request Guidelines

All contributors must submit their changes via pull requests. If you're not familiar with pull requests, please read [the GitHub documentation](https://help.github.com/en/articles/about-pull-requests). Following are a few guidelines and expectations for submitting a PR.

1. Fork the repo that you want to contribute to ([ocsf-schema](https://github.com/ocsf/ocsf-schema), [ocsf-docs](https://github.com/ocsf/ocsf-docs), [ocsf-server](https://github.com/ocsf/ocsf-server)).
2. Make desired changes in the forked repo, test if [everything works as expected](#verifying-the-changes) and is error-free, a local instance of the [ocsf-server](https://github.com/ocsf/ocsf-server) would be essential.
3. Push the changes to the forked repo.
4. Create a **Pull Request** to merge changes into the main repo, request at least 3 approvers.
5. Follow the guidelines in the PR template.
1. Limit the number of commits in a single PR to aid reviewers, be as specific with the change as possible. A single PR **must** contain related changes.
2. Each commit must include a DCO [Developer's Certificate of Origin](#developers-certificate-of-origin-11).
3. Add an entry to the `Unreleased` section in [CHANGELOG.md](https://github.com/ocsf/ocsf-schema/blob/main/CHANGELOG.md).
3. Describe your change in as much detail as possible.
5. Confirm that you have tested the changes, and the server run was error free.
6. Check the Preview tab to ensure everything looks as expected.
7. Once the PR is ready, add relevant labels, request approvers and submit it.
8. Resolve any github action failures, warnings reported for your pull request, and stay involved in the conversation.
6. Thank you for your contribution!

* * *

### Key OCSF Terminology

1. **Field**: A field is a unique identifier name for a piece of data contained in OCSF. Each field also designates a corresponding data_type.
2. **Object**: An object is a collection of contextually related fields and other objects. It is also a data_type in OCSF.
Expand Down Expand Up @@ -231,23 +252,7 @@ Sample error messages -
```
Address the errors before submitting the changes, your server run should be completely error free.

### Pull Request Guidelines

All contributors should submit their changes via pull requests. If you're not familiar with pull requests, please read [the GitHub documentation](https://help.github.com/en/articles/about-pull-requests). Following are a few guidelines and expectations for submitting a PR.

1. Fork the repo that you want to contribute to ([ocsf-schema](https://github.com/ocsf/ocsf-schema), [ocsf-docs](https://github.com/ocsf/ocsf-docs), [ocsf-server](https://github.com/ocsf/ocsf-server))
2. Make desired changes in the forked repo, test if everything works as expected and is error-free, a local instance of the [ocsf-server](https://github.com/ocsf/ocsf-server) would be essential.
3. Push the changes to the forked repo
4. Create a **Pull Request** to merge changes into the main repo, request at least 3 approvers.
1. Limit the number of commits in a single PR to aid reviewers, be as specific with the change as possible. A single PR must contain related changes.
2. Each commit must include a DCO [Developer's Certificate of Origin](#developers-certificate-of-origin-11)
3. Describe your change in as much detail as possible.
4. Confirm that you have tested the changes, and the server run was error free.
5. Check the Preview tab to ensure everything looks as expected.
6. Once the PR is ready, add relevant labels, request approvers and submit it.
7. Pay attention to any automated CI failures, warnings reported in the pull request, and stay involved in the conversation.

* * *
***

### Developer's Certificate of Origin 1.1

Expand Down

0 comments on commit 596fd07

Please sign in to comment.