Skip to content

Added a workflow for code autoformatting#43

Open
jrzkaminski wants to merge 6 commits intomainfrom
black-workflow
Open

Added a workflow for code autoformatting#43
jrzkaminski wants to merge 6 commits intomainfrom
black-workflow

Conversation

@jrzkaminski
Copy link
Contributor

@jrzkaminski jrzkaminski commented Dec 24, 2024

This pull request includes several changes across multiple files to improve code formatting, consistency, and functionality. The main changes involve the addition of a new GitHub Actions workflow for linting, updates to existing workflows, and numerous formatting adjustments in Python example files.

GitHub Actions Workflows:

  • .github/workflows/black.yml: Added a new workflow for linting using psf/black.
  • .github/workflows/integration-build.yml and .github/workflows/unit-build.yml: Corrected the naming convention for llm_api dependencies and tests. [1] [2]

Code Formatting:

  • examples/connector_creator_usage_example.py, examples/llama31_usage_example.py, examples/metrics_usage_examples.py, examples/real_world/chemical_multi_agent_system/chain.py, examples/real_world/chemical_multi_agent_system/prompting.py, and examples/real_world/chemical_multi_agent_system/tools.py: Applied consistent formatting, such as breaking long lines and using double quotes for strings. [1] [2] [3] [4] [5] [6]

Functionality Improvements:

Minor Adjustments:

@jrzkaminski jrzkaminski requested a review from nicl-nno December 24, 2024 13:29
@jrzkaminski jrzkaminski self-assigned this Dec 24, 2024
Copy link
Contributor

@DRMPN DRMPN left a comment

Choose a reason for hiding this comment

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

Я для Федота делал action с black по его документации.
Мне просто интересно, этот action лучше или хуже, чем официальный?

@jrzkaminski
Copy link
Contributor Author

Я для Федота делал action с black по его документации. Мне просто интересно, этот action лучше или хуже, чем официальный?

Возможно, что лучше. А возможно, что ноль отличий, не знаю, если честно

This was linked to issues Mar 18, 2025
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.

Multiple non-PEP hyphens usage Non-existant import

3 participants