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

[DRAFT] Enable GitHub Actions: Set up linters using GHA #776

Closed
wants to merge 6 commits into from

Conversation

sweep-nightly[bot]
Copy link
Contributor

@sweep-nightly sweep-nightly bot commented Jul 25, 2023

Description

This PR enables GitHub Actions and sets up linters using GHA based on the primary language of the repository.

Changes Made

  • Modified on_pull_request.py to detect the merge of the "Enable Github Actions" PR and create a new GitHub issue.
  • Modified create_issue.py to create a new GitHub issue with the appropriate title and content.
  • Modified repo.py to determine the primary language of the repository.
  • Modified issue_content_generator.py to generate the appropriate issue content based on the primary language of the repository.

Checklist

  • Tested the changes locally
  • Updated the documentation
  • Added unit tests
  • Verified the changes against the requirements

Additional Notes

Please review and merge this PR to enable GitHub Actions and set up linters using GHA.

Fixes #768.

To checkout this PR branch, run the following command in your terminal:

git checkout sweep/enable-gha-linters

@sweep-nightly sweep-nightly bot added the sweep Assigns Sweep to an issue or pull request. label Jul 25, 2023
@vercel
Copy link

vercel bot commented Jul 25, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sweep-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 25, 2023 3:43am

Copy link
Contributor Author

@sweep-nightly sweep-nightly bot left a comment

Choose a reason for hiding this comment

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

The code changes in the pull request are generally good, but there's a small issue that needs to be addressed in all the files. Each file is missing a newline at the end. This is a common standard that helps ensure compatibility with various tools that might not handle the last line correctly if it doesn't end with a newline.

Here are the specific changes that need to be made:

  • Add a newline at the end of the file sweepai/handlers/create_issue.py
  • Add a newline at the end of the file sweepai/handlers/on_pull_request.py
  • Add a newline at the end of the file sweepai/issue_content_generator.py
  • Add a newline at the end of the file sweepai/repo.py

Please make these changes and update the pull request. Keep up the good work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sweep Assigns Sweep to an issue or pull request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sweep(slow): on enable GitHub actions, make Sweep tickets to set up linters using GHA
0 participants