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

adding code reviews #178

Merged
merged 65 commits into from
May 31, 2024
Merged

adding code reviews #178

merged 65 commits into from
May 31, 2024

Conversation

seanctech
Copy link
Member

@seanctech seanctech commented Nov 15, 2023

Contributor Comments

This PR adds OpenAI security-focused code reviews against GitHub pull requests.

Pull Request Checklist

Thank you for submitting a contribution to our project!

In order to streamline the review of your contribution we ask that you review and comply with the below requirements:

  • Rebase your branch against the latest commit of the target branch.
  • If you are adding a dependency, please explain how it was chosen.
  • If manual testing is needed in order to validate the changes, provide a testing plan and the expected results.
  • If there is an issue associated with your Pull Request, link the issue to the PR.
  • Validate that documentation is accurate and aligned to any project updates or additions.

Don't forget our more detailed contribution guidelines
here.

entrypoint.sh Outdated Show resolved Hide resolved
ChiefHolland and others added 11 commits March 19, 2024 14:37
Signed-off-by: Seiso Automation <[email protected]>
Co-authored-by: ChiefHolland <[email protected]>

    pick 1259b5d Bump version: 2024.01.04 → 2024.04.01
    pick 77fde21 chore(deps): bump black from 23.12.1 to 24.3.0 (#185)
Bumps [black](https://github.com/psf/black) from 23.12.1 to 24.3.0.
- [Release notes](https://github.com/psf/black/releases)
- [Changelog](https://github.com/psf/black/blob/main/CHANGES.md)
- [Commits](psf/black@23.12.1...24.3.0)

---
updated-dependencies:
- dependency-name: black
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    pick 7ae118c update Pipfile.lock
    pick 861fdc3 Adding code review call and updating prompt

    pick 6f02d8b update Pipfile.lock
    pick 020471e Adding dependencies
    pick 861fdc3 Adding code review call and updating prompt
    pick a358296 Adding dependencies

    pick 020471e Adding dependencies
    pick 0cf98ef Cleaning up Dockerfile
    pick a5b29da Updating code
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Salacious has reviewed your code, please see inline comments.

List of skipped files:
['.github/CONTRIBUTING.md', '.github/workflows/commit.yml', '.pre-commit-config.yaml', 'Dockerfile', 'Pipfile', 'Pipfile.lock', 'README.md', 'Taskfile.yml', 'code_review.py', 'code_reviews/init.py', 'code_reviews/config.py', 'code_reviews/constants.py', 'entrypoint.sh', 'package-lock.json', 'setup.cfg']

package.json Show resolved Hide resolved
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Salacious has reviewed your code, please see inline comments.

List of skipped files:
['Pipfile.lock', 'code_review.py', 'package-lock.json']

.github/CONTRIBUTING.md Show resolved Hide resolved
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Salacious has reviewed your code, please see inline comments.

List of skipped files:
['Pipfile.lock', 'code_review.py', 'package-lock.json']

.github/CONTRIBUTING.md Show resolved Hide resolved
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Salacious has reviewed your code, please see inline comments.

List of skipped files:
['Pipfile.lock', 'code_review.py', 'package-lock.json']

.github/CONTRIBUTING.md Show resolved Hide resolved
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Salacious has reviewed your code, please see inline comments.

List of skipped files:
['Pipfile.lock', 'code_review.py', 'package-lock.json']

.github/CONTRIBUTING.md Show resolved Hide resolved
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Salacious has reviewed your code, please see inline comments.

List of skipped files:
['Pipfile.lock', 'code_review.py', 'package-lock.json']

.github/CONTRIBUTING.md Show resolved Hide resolved
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Salacious has reviewed your code, please see inline comments.

List of skipped files:
['.github/CONTRIBUTING.md', 'Pipfile.lock', 'code_review.py', 'package-lock.json']

.github/workflows/commit.yml Show resolved Hide resolved
@ChiefHolland ChiefHolland marked this pull request as ready for review May 31, 2024 16:14
@ChiefHolland ChiefHolland enabled auto-merge (squash) May 31, 2024 16:16
@ChiefHolland ChiefHolland merged commit e40e851 into main May 31, 2024
2 checks passed
@ChiefHolland ChiefHolland deleted the feat/salacious branch May 31, 2024 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants