sweep icon indicating copy to clipboard operation
sweep copied to clipboard

[Sweep GHA Fix] The GitHub Actions run failed with...

Open sweep-nightly[bot] opened this issue 1 year ago • 2 comments

Description

This pull request includes changes to the sweepai/api.py, sweepai/handlers/on_check_suite.py, and sweepai/utils/github_utils.py files. It updates the worker function in api.py, the clean_logs function in on_check_suite.py, and the get_token, get_installation_id, clone, and get_commit_history functions in github_utils.py.

Summary

  • Updated the worker function in api.py to use the make_pr function instead of the stack_pr function and added new parameters.
  • Modified the clean_logs function in on_check_suite.py to return the cleaned logs and user message separately.
  • Updated the get_token, get_installation_id, clone, and get_commit_history functions in github_utils.py to improve error handling and logging.

sweep-nightly[bot] avatar Jan 12 '24 21:01 sweep-nightly[bot]

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 Jan 12, 2024 9:35pm

vercel[bot] avatar Jan 12 '24 21:01 vercel[bot]

Apply Sweep Rules to your PR?

  • [ ] Apply: We should use loguru for error logging. If the log is inside an exception, use logger.exception to add tracebacks, where logger is imported from loguru. Use f-strings for string formatting in logger calls (e.g. logger.info(f'Hello {name}') instead of logger.info('Hello {name}', name=name)).
  • [ ] Apply: There should be no debug log or print statements in production code.
  • [ ] Apply: All functions should have parameters and output annotated with type hints. Use list, tuple and dict instead of typing.List, typing.Tuple and typing.dict.
  • [ ] Apply: Leftover TODOs in the code should be handled.
  • [ ] Apply: All new business logic should have corresponding unit tests in the same directory. For example, sweepai/api_test.py tests sweepai/api.py. Use unittest and unittest.mock as required.
  • [ ] Apply: Any clearly inefficient or repeated code should be optimized or refactored.
  • [ ] Apply: Remove any comments before code that are obvious. For example # this prints hello world; print('hello world').

This is an automated message generated by Sweep AI.

sweep-nightly[bot] avatar Jan 12 '24 21:01 sweep-nightly[bot]