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... #9

Open
wants to merge 3 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 from the format and format-check commands.
  • Updated the openoligo/api/helpers.py file to replace the SynthesisQueue model with SynthesisTask in various functions.
  • 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 replace the SynthesisQueue model with SynthesisTask in the update_task_status, set_started_now, set_completed_now, set_failed_now, set_task_in_progress, set_log_file, get_log_file, set_org_id, create_new_reactant, and get_next_task functions.
  • Updated the openoligo/hal/board.py file to replace the NoSuchPinInPinout exception with NoSuchPinInPinoutError.
  • Updated the openoligo/hal/instrument.py file to replace the OneSourceException and OneDestinationException exceptions with OneSourceError and OneDestinationError.
  • Updated the openoligo/hal/platform.py file to remove the unused is_bb function.
  • Updated the openoligo/hal/types.py file to replace the NoSuchPinInPinout exception with NoSuchPinInPinoutError.
  • Updated the openoligo/scripts/runner.py file to replace the SynthesisQueue model with SynthesisTask in the worker function.
  • Updated the openoligo/scripts/server.py file to replace the SynthesisQueue model with SynthesisTask in the add_a_task_to_synthesis_queue, get_all_tasks_in_synthesis_queue, update_a_synthesis_task, delete_synthesis_task_by_id, and get_task_by_id functions.
  • Updated the openoligo/utils/singleton.py file to remove the unused Singleton metaclass.
  • Updated the openoligo/utils/wait.py file to remove the unused imports.
  • Updated the pyproject.toml file to remove the unused redis and types-redis dependencies.
  • Updated the setup.py file to add a description for the library.
  • Updated the tests/api/test_helpers.py file to replace the SynthesisQueue model with SynthesisTask in the tests.
  • Updated the tests/api/test_models.py file to replace the SynthesisQueue model with SynthesisTask in the tests.
  • Updated the tests/api/test_server.py file to add tests for the get_all_tasks_in_synthesis_queue and get_task_by_id functions.
  • Updated the tests/hal/test_devices.py file to replace the OneDestinationException and OneSourceException exceptions with OneDestinationError and OneSourceError.
  • Updated the tests/hal/test_instrument.py file to replace the OneDestinationException and OneSourceException exceptions with OneDestinationError and OneSourceError.
  • Updated the tests/test_examples.py file to remove the unused import.

@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