AutoGPT icon indicating copy to clipboard operation
AutoGPT copied to clipboard

ci: move to Python 3.10 & improve workflow

Open Pwuts opened this issue 2 years ago • 2 comments

Background

Workflow was updated but now runs twice. Also runs with Python 3.8 which is inconsistent with README (#2342). pull_request_target is both risky and unnecessary: CI will already run on push in the fork.

Changes

  • Split build workflow into lint and test workflows
  • Run all checks even if some fail
  • Fix double PR runs
  • Adopted #2343

~~Documentation~~

~~Test Plan~~

PR Quality Checklist

  • [ ] My pull request is atomic and focuses on a single change.
  • [ ] I have thoroughly tested my changes with multiple different prompts.
  • [x] I have considered potential risks and mitigations for my changes.
  • [x] I have documented my changes clearly and comprehensively.
  • [x] I have not snuck in any "extra" small tweaks changes

Pwuts avatar Apr 18 '23 12:04 Pwuts

Related: https://github.com/Significant-Gravitas/Auto-GPT/pull/2267

aperullo avatar Apr 18 '23 14:04 aperullo

Don't forget to update .github/workflows/benchmark.yml and .devcontainer/Dockerfile, see #2107

Androbin avatar Apr 18 '23 17:04 Androbin

PR for #2424 pt 1: "shirt sizing" is ready pending this pull request being merged into master on Pwuts/Auto-GPT/tree/ci/shirt-sizing

ntindle avatar Apr 18 '23 20:04 ntindle