Closed sweep-ai[bot] closed 12 months ago
The command:
Run pipenv run black --check .
yielded the following error:
`##[error]Process completed with exit code 123.
[36;1mdocker build . -t pulling_ace:test[0m shell: /usr/bin/bash -e {0}
[python-deps 4/7] COPY pulling_ace /app/pulling_ace:
Dockerfile:19
17 |
18 | # Copy the pulling_ace directory into the Docker image 19 | >>> COPY pulling_ace /app/pulling_ace 20 |
21 | # Install pipenv and compilation dependenciesERROR: failed to solve: failed to compute cache key: failed to calculate checksum of ref 5ee0a90d-9ba7-4120-a52c-7d8898402fd7::z4mgtm906fgrr131e5ug3zwll: "/pulling_ace": not found
[error]Process completed with exit code 1.`
Here are the logs:
error: cannot format pulling_ace/cli.py: Cannot parse: 39:4: run_injections(args.model_type, args.model_name, args.probes) would reformat setup.py
Oh no! 💥 💔 💥 1 file would be reformatted, 18 files would be left unchanged, 1 file would fail to reformat.
Done.
PR Feedback (click)
[ ] 👍 Sweep Did Well
[ ] 👎 Sweep Needs Improvement
I created this PR to address this rule:
"Code should follow a consistent coding style and formatting."
Description
This PR addresses the inconsistency in coding style and formatting as reported in the issue. It includes the following changes:
Removed the extra space before the function call "run_injections" in the "cli.py" file to maintain consistent formatting.
Updated the console script entry point in the "setup.py" file to follow a consistent naming convention, changing it to "pulling-ace-cli".
Summary of Changes