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.

Sign up for GitHub

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

Jump to bottom

Format files with new ruff 0.9.0 #853

Merged
merged 1 commit into from
Jan 9, 2025
Merged

Format files with new ruff 0.9.0 #853

merged 1 commit into from
Jan 9, 2025

Conversation

Zakaria-Latif
Copy link
Member

No description provided.

@Zakaria-Latif Zakaria-Latif self-assigned this Jan 9, 2025
@Zakaria-Latif Zakaria-Latif requested a review from a team as a code owner January 9, 2025 16:43
ostorlab-ai-pr-review[bot]
ostorlab-ai-pr-review bot reviewed Jan 9, 2025
View reviewed changes
Copy link

@ostorlab-ai-pr-review ostorlab-ai-pr-review bot left a comment

Choose a reason for hiding this comment

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

AI Bot Review comment

src/ostorlab/agent/message/serializer.py Show resolved Hide resolved
src/ostorlab/agent/schema/validator.py Show resolved Hide resolved
src/ostorlab/apis/runners/authenticated_runner.py Show resolved Hide resolved
src/ostorlab/apis/runners/authenticated_runner.py Show resolved Hide resolved
src/ostorlab/apis/runners/login_runner.py Show resolved Hide resolved
src/ostorlab/apis/runners/public_runner.py Show resolved Hide resolved
src/ostorlab/cli/ci_scan/run/assets/link.py Show resolved Hide resolved
src/ostorlab/cli/ci_scan/run/assets/mobile.py Show resolved Hide resolved
src/ostorlab/cli/ci_scan/run/run.py Show resolved Hide resolved
src/ostorlab/runtimes/cloud/runtime.py Show resolved Hide resolved
@amine3 amine3 merged commit 13329ac into main Jan 9, 2025
10 checks passed
@Zakaria-Latif Zakaria-Latif deleted the ruff-format branch January 9, 2025 16:57
@codecov Codecov
Copy link

codecov bot commented Jan 9, 2025

Codecov Report

Attention: Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 62.85%. Comparing base (a973770) to head (d68f379).
Report is 12 commits behind head on main.

Files with missing lines Patch % Lines
src/ostorlab/agent/message/serializer.py 0.00% 1 Missing ⚠️
Additional details and impacted files 
@@           Coverage Diff           @@
##             main     #853   +/-   ##
=======================================
  Coverage   62.85%   62.85%           
=======================================
  Files         342      342           
  Lines       14566    14566           
=======================================
  Hits         9156     9156           
  Misses       5410     5410

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ostorlab-ai-pr-review ostorlab-ai-pr-review[bot] ostorlab-ai-pr-review[bot] left review comments

@adnaneserrar adnaneserrar adnaneserrar approved these changes

@ohachimOs ohachimOs ohachimOs approved these changes

@benyissa benyissa benyissa approved these changes

@oussamaessaji oussamaessaji oussamaessaji approved these changes

@amine3 amine3 amine3 approved these changes

Assignees

@Zakaria-Latif Zakaria-Latif

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@Zakaria-Latif @amine3 @oussamaessaji @benyissa @ohachimOs @adnaneserrar