Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot commented Apr 12, 2025

Preface From AJ

This fixes a bunch of improvements needed by the latest version of ruff.

Should be a no-op apart from formatting/linting improvements.

Summary From Devin

Updated ruff to the latest version (0.11.5) and ran auto-format and auto-fix.

Requested by: Aaron ("AJ") Steers ([email protected])
Link to Devin run: https://app.devin.ai/sessions/b7b8550430a340bbb4e58b717ad8ac45

devin-ai-integration bot and others added 2 commits April 12, 2025 06:46
@devin-ai-integration
Copy link
Contributor Author

Original prompt from Aaron:

@Devin - In the CDK repo, bump to the latest version of ruff and then auto-format and auto-fix lint issue. Commit the result in a new PR.

@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@github-actions github-actions bot added the chore label Apr 12, 2025
@aaronsteers aaronsteers requested a review from maxi297 April 12, 2025 06:53
@aaronsteers aaronsteers changed the title chore: bump ruff to 0.11.5 chore: bump ruff to 0.11.5, apply auto-fixes Apr 12, 2025
@aaronsteers aaronsteers requested review from bnchrch and lazebnyi April 12, 2025 06:57
Copy link
Contributor

@aaronsteers aaronsteers left a comment

Choose a reason for hiding this comment

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

No changes requested - this is a test. (Ignore.)

@aaronsteers aaronsteers dismissed their stale review April 13, 2025 00:39

Implemented the requested changes. Thanks!! 🙏

Copy link
Contributor

@maxi297 maxi297 left a comment

Choose a reason for hiding this comment

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

I assume that the changes are only formatting change except from the ruff version change from 0.8.3 to 0.11.5. If not or if you want me to check something specifically, please let me know

@aaronsteers
Copy link
Contributor

Thanks, @maxi297! Yes, this is just auto-formatting changes, nothing changed otherwise.

@aaronsteers aaronsteers merged commit fa7f2e6 into main Apr 14, 2025
30 checks passed
@aaronsteers aaronsteers deleted the devin/1744440307-update-ruff branch April 14, 2025 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants