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

Update ruff to v0.1.3 #2235

Merged
merged 1 commit into from
Oct 31, 2023
Merged

Conversation

vshampor
Copy link
Contributor

Changes

As stated in the title

Reason for changes

Fixes the following message (spawned when ruff finds an error):

panicked at 'byte index 1024 is not a char boundary; it is inside '█' (bytes 1023..1026) of `# Copyright (c) 2023 Intel Corporation
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#      http://www.apache.org/licenses/LICE`[...]', /home/runner/work/ruff/ruff/crates/ruff_source_file/src/locator.rs:382:10
Backtrace:    0: <unknown>
   1: <unknown>
   2: <unknown>

Related tickets

N/A

Tests

pre-commit linters

@vshampor vshampor requested a review from a team as a code owner October 31, 2023 12:19
@codecov
Copy link

codecov bot commented Oct 31, 2023

Codecov Report

Merging #2235 (81dfe34) into develop (c732de0) will increase coverage by 54.18%.
The diff coverage is n/a.

Additional details and impacted files
@@             Coverage Diff              @@
##           develop    #2235       +/-   ##
============================================
+ Coverage    36.40%   90.58%   +54.18%     
============================================
  Files          486      486               
  Lines        43483    43521       +38     
============================================
+ Hits         15829    39425    +23596     
+ Misses       27654     4096    -23558     
Flag Coverage Δ
ONNX 33.69% <ø> (?)
OPENVINO 38.33% <ø> (?)
TENSORFLOW 30.03% <ø> (?)
TORCH 62.75% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 354 files with indirect coverage changes

@vshampor
Copy link
Contributor Author

TF precommit checks fail due to a known unrelated issue.

@vshampor vshampor merged commit b2a242b into openvinotoolkit:develop Oct 31, 2023
9 of 10 checks passed
@vshampor vshampor deleted the update_ruff branch January 9, 2024 14:01
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.

2 participants