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

Run pre-commit #1741

Merged
merged 1 commit into from
Dec 29, 2024
Merged

Run pre-commit #1741

merged 1 commit into from
Dec 29, 2024

Conversation

NolanTrem
Copy link
Collaborator

@NolanTrem NolanTrem commented Dec 29, 2024

Important

Remove trailing spaces in llms.txt and test_collections.py.

  • Whitespace Cleanup:
    • Remove trailing spaces in llms.txt and test_collections.py.

This description was created by Ellipsis for e2fc53b. It will automatically update as commits are pushed.

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to e2fc53b in 11 seconds

More details
  • Looked at 1307 lines of code in 2 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. py/tests/unit/test_collections.py:176
  • Draft comment:
    Trailing spaces were removed throughout the file. This is a good practice for maintaining clean code.
  • Reason this comment was not posted:
    Confidence changes required: 10%
    The PR primarily focuses on removing trailing spaces. The test file contains multiple instances of trailing spaces that were removed. This is a good practice for maintaining clean code.

Workflow ID: wflow_rddVu5UKzFH0zrLl


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@NolanTrem NolanTrem merged commit a2108ef into main Dec 29, 2024
2 of 3 checks passed
@NolanTrem NolanTrem deleted the Nolan/FixPreCommit branch December 29, 2024 05:14
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.

1 participant