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

ref: Fix ruff rules in preview mode #4039

Merged
merged 1 commit into from
Oct 8, 2024

Conversation

cbornet
Copy link
Collaborator

@cbornet cbornet commented Oct 7, 2024

Fix ruff rules in preview mode.
Note: we don't use preview mode in CI (for good reason)

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Oct 7, 2024
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-4039.dmtpw4p5recq1.amplifyapp.com

@cbornet cbornet force-pushed the ruff-rules-preview branch from 8a72c66 to 9d0970a Compare October 7, 2024 10:54
#! This part here is clunky but we need it like this for
#! backwards compatibility. We can change how prompt component
#! works and then update this later
# ! This part here is clunky but we need it like this for
Copy link
Collaborator Author

@cbornet cbornet Oct 7, 2024

Choose a reason for hiding this comment

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

Is the comment syntax #! used by something ? (conflicts with shebang btw) ?

Copy link
Contributor

Choose a reason for hiding this comment

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

Not really. I use https://marketplace.visualstudio.com/items?itemName=aaron-bond.better-comments and that makes it pop a bit more but not necessary. It is just to make sure people pay attention to certain comments more than others.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Does it still highlight if you have a space before the ! ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think it should work and is recommended to have the space: aaron-bond/better-comments#8 (comment)

Copy link
Contributor

Choose a reason for hiding this comment

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

Go for it. Thanks for the heads up!

@cbornet cbornet requested a review from ogabrielluiz October 7, 2024 11:00
@cbornet cbornet force-pushed the ruff-rules-preview branch 3 times, most recently from 05d4bf8 to 16de7e1 Compare October 7, 2024 13:19
@cbornet
Copy link
Collaborator Author

cbornet commented Oct 7, 2024

Added support for EXE rule as it passes once # ! is used instead of #!

@cbornet cbornet force-pushed the ruff-rules-preview branch from 16de7e1 to 7c41224 Compare October 7, 2024 21:25
Copy link
Contributor

@ogabrielluiz ogabrielluiz left a comment

Choose a reason for hiding this comment

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

LGTM

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 8, 2024
@cbornet cbornet force-pushed the ruff-rules-preview branch from 7c41224 to 11fb7c8 Compare October 8, 2024 19:17
@cbornet cbornet force-pushed the ruff-rules-preview branch from 11fb7c8 to 7403f2b Compare October 8, 2024 19:23
@cbornet cbornet enabled auto-merge (squash) October 8, 2024 19:27
@cbornet cbornet merged commit da6b5ea into langflow-ai:main Oct 8, 2024
29 checks passed
@cbornet cbornet deleted the ruff-rules-preview branch October 8, 2024 19:31
diogocabral pushed a commit to headlinevc/langflow that referenced this pull request Nov 26, 2024
Fix some ruff rules from preview mode
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants