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

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

Open
wants to merge 6 commits into
base: betterapi
Choose a base branch
from

Conversation

sweep-ai[bot]
Copy link

@sweep-ai sweep-ai bot commented Jan 10, 2024

Description

This pull request includes the following changes:

  • Updated the .flake8 file to increase the max-line-length from 100 to 200.
  • Updated the Makefile to remove the @ symbol before the commands in the format and format-check targets.
  • Updated the openoligo/api/helpers.py file to change the import statement for the SynthesisTask model.
  • Updated the openoligo/api/helpers.py file to change the function names and update the database queries to use the SynthesisTask model instead of the SynthesisQueue model.
  • Updated the openoligo/api/models.py file to add new enums (InstrumentHealth, Backbone, Nucleobase, Sugar, SolidSupport) and a new dataclass (NucleobaseInfo).
  • Updated the openoligo/api/models.py file to change the import statement for the SynthesisTask model.
  • Updated the openoligo/api/models.py file to add new fields and validators to the Nucleotide model.
  • Updated the openoligo/api/models.py file to change the name of the SynthesisQueue model to SynthesisTask and update the field names and types accordingly.
  • Updated the openoligo/api/models.py file to change the import statement for the SynthesisTask model.
  • Updated the openoligo/api/redis.py file to remove the unused imports and update the get_redis function to return the red object directly.
  • Updated the openoligo/hal/board.py file to change the import statement for the NoSuchPinInPinout exception.
  • Updated the openoligo/hal/instrument.py file to change the import statement for the OneDestinationException and OneSourceException exceptions.
  • Updated the openoligo/hal/platform.py file to change the import statement for the NoSuchPinInPinout exception.
  • Updated the openoligo/hal/types.py file to change the import statement for the NoSuchPinInPinout exception and update the names of the OneDestinationException and OneSourceException exceptions.
  • Updated the openoligo/scripts/runner.py file to change the import statement for the SynthesisQueue model.
  • Updated the openoligo/scripts/server.py file to change the import statement for the SynthesisQueue model and update the /health endpoint to return the InstrumentHealth.OPERATIONAL status.
  • Updated the openoligo/scripts/server.py file to change the import statement for the SynthesisQueue model and update the /queue endpoints to use the SynthesisTask model.
  • Updated the openoligo/utils/singleton.py file to remove the unused import.
  • Updated the openoligo/utils/wait.py file to remove the unused imports.
  • Updated the pyproject.toml file to remove the unused dependencies and add the ruff package.
  • Updated the setup.py file to add a description and update the version number.
  • Updated the tests/api/test_helpers.py file to change the import statement for the SynthesisQueue model.
  • Updated the tests/api/test_models.py file to change the import statement for the SynthesisQueue model and update the tests to use the SynthesisTask model.
  • Updated the tests/api/test_server.py file to change the import statement for the SynthesisQueue model.
  • Updated the tests/hal/test_instrument.py file to change the import statement for the OneDestinationException and OneSourceException exceptions.

@sweep-ai sweep-ai bot mentioned this pull request Jan 10, 2024
Copy link
Author

sweep-ai bot commented Jan 10, 2024

Apply Sweep Rules to your PR?

  • Apply: All new business logic should have corresponding unit tests.
  • Apply: Refactor large functions to be more modular.
  • Apply: Add docstrings to all functions and file headers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants