Skip to content

Modernize packaging#28

Merged
mshriver merged 3 commits into
mainfrom
mypy-intro
Sep 18, 2025
Merged

Modernize packaging#28
mshriver merged 3 commits into
mainfrom
mypy-intro

Conversation

@mshriver
Copy link
Copy Markdown
Contributor

@mshriver mshriver commented Sep 17, 2025

slingshot the repo into the future.

No functional changes intended here. Local testing showing consistent API results for uploading through pytest-ibutsu.

apply pre-commit, with mypy

limited due to code generation
Copilot AI review requested due to automatic review settings September 17, 2025 14:27
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR modernizes the packaging and code style of the ibutsu_client codebase, focusing on Python code formatting improvements and import reorganization.

  • Standardizes string quotes from single to double quotes throughout the codebase
  • Reorganizes imports and removes unnecessary blank lines
  • Improves code formatting for better readability and consistency

Reviewed Changes

Copilot reviewed 65 out of 133 changed files in this pull request and generated no comments.

File Description
Multiple model files Updated string quotes to double quotes, reorganized imports, improved formatting
ibutsu_client/exceptions.py Modernized string formatting with f-strings, improved class inheritance syntax
ibutsu_client/configuration.py Updated string quotes, improved formatting, modernized f-string usage
ibutsu_client/apis/init.py Removed unnecessary blank line

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@mshriver mshriver added the enhancement New feature or request label Sep 18, 2025
@mshriver mshriver merged commit d120343 into main Sep 18, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants