Skip to content

chore(deps): lock file maintenance #1121

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
Merged
merged 1 commit into from
May 6, 2025
Merged

chore(deps): lock file maintenance #1121

merged 1 commit into from
May 6, 2025
+179 −176

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented May 5, 2025 •
edited
Loading

This PR contains the following updates:

Update Change
lockFileMaintenance All locks refreshed

🔧 This Pull Request updates lock files to use the latest dependency versions.

Configuration

📅 Schedule: Branch creation - Between 12:00 AM and 03:59 AM, only on Monday ( * 0-3 * * 1 ) (UTC), Automerge - Between 12:00 AM and 03:59 AM ( * 0-3 * * * ) (UTC).

🚦 Automerge: Enabled.

♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate renovate bot requested review from csjall and dixonjoel as code owners May 5, 2025 02:17
@renovate renovate bot enabled auto-merge (squash) May 5, 2025 02:17
@renovate renovate bot requested a review from bkeryan as a code owner May 5, 2025 02:17
@github-actions GitHub Actions
Copy link

github-actions bot commented May 5, 2025 •
edited
Loading

Test Results

    52 files  +  8      52 suites  +8   1h 17m 0s ⏱️ + 7m 2s
   709 tests ±  0     709 ✅ +  1      0 💤 ±0  0 ❌  - 1 
21 420 runs  +248  20 344 ✅ +249  1 076 💤 ±0  0 ❌  - 1 

Results for commit aae8281. ± Comparison against base commit 65d348e.

♻️ This comment has been updated with latest results.

@bkeryan
Copy link
Collaborator

bkeryan commented May 5, 2025

@dixonjoel I'll update pep8-naming in a separate PR.

Run poetry run ni-python-styleguide lint
./ni_measurement_plugin_sdk_service/session_management/_reservation.py:77:2: N808 type variable name '_T' should use CapWords convention and an optional '_co' or '_contra' suffix
./ni_measurement_plugin_sdk_service/session_management/_reservation.py:77:2: N808 type variable name '_T' should use CapWords convention and an optional '_co' or '_contra' suffix
./ni_measurement_plugin_sdk_service/grpc/loggers.py:356:2: N808 type variable name '_T' should use CapWords convention and an optional '_co' or '_contra' suffix
./ni_measurement_plugin_sdk_service/grpc/loggers.py:356:2: N808 type variable name '_T' should use CapWords convention and an optional '_co' or '_contra' suffix
./tests/utilities/connection_subset.py:[13](https://github.com/ni/measurement-plugin-python/actions/runs/14827763686/job/41623424273?pr=1121#step:9:14):2: N808 type variable name '_T' should use CapWords convention and an optional '_co' or '_contra' suffix
./tests/utilities/connection_subset.py:13:2: N808 type variable name '_T' should use CapWords convention and an optional '_co' or '_contra' suffix
./tests/utilities/connection_subset.py:[14](https://github.com/ni/measurement-plugin-python/actions/runs/14827763686/job/41623424273?pr=1121#step:9:15):2: N808 type variable name '_TMultiplexer' should use CapWords convention and an optional '_co' or '_contra' suffix
./tests/utilities/connection_subset.py:14:2: N808 type variable name '_TMultiplexer' should use CapWords convention and an optional '_co' or '_contra' suffix
Error: Process completed with exit code 255.

@renovate renovate bot force-pushed the users/renovate/lock-file-maintenance branch from cca3856 to aae8281 Compare May 5, 2025 23:35
@renovate renovate bot merged commit 5a68ab3 into main May 6, 2025
17 checks passed
@renovate renovate bot deleted the users/renovate/lock-file-maintenance branch May 6, 2025 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@csjall csjall Awaiting requested review from csjall csjall is a code owner

@dixonjoel dixonjoel Awaiting requested review from dixonjoel dixonjoel is a code owner

@bkeryan bkeryan Awaiting requested review from bkeryan bkeryan is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@bkeryan