Skip to content

Comments

Use __future__.annotations#345

Merged
JacobHayes merged 1 commit intogoldenfrom
future-annotations
Mar 20, 2023
Merged

Use __future__.annotations#345
JacobHayes merged 1 commit intogoldenfrom
future-annotations

Conversation

@JacobHayes
Copy link
Member

Description

I previously removed from __future__ import annotations in #116, but the original pydantic bug has been fixed. This adds the import back and tidies the manually stringized hints.

Signed-off-by: Jacob Hayes <[email protected]>
@codecov
Copy link

codecov bot commented Mar 20, 2023

Codecov Report

Merging #345 (c54b0a7) into golden (18a041a) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            golden      #345   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           43        43           
  Lines         2799      2829   +30     
  Branches       710       710           
=========================================
+ Hits          2799      2829   +30     
Impacted Files Coverage Δ
src/arti/annotations/__init__.py 100.00% <100.00%> (ø)
src/arti/artifacts/__init__.py 100.00% <100.00%> (ø)
src/arti/executors/local.py 100.00% <100.00%> (ø)
src/arti/formats/__init__.py 100.00% <100.00%> (ø)
src/arti/formats/json.py 100.00% <100.00%> (ø)
src/arti/formats/pickle.py 100.00% <100.00%> (ø)
src/arti/internal/__init__.py 100.00% <100.00%> (ø)
src/arti/internal/dispatch.py 100.00% <100.00%> (ø)
src/arti/internal/patches.py 100.00% <100.00%> (ø)
src/arti/internal/type_hints.py 100.00% <100.00%> (ø)
... and 20 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@JacobHayes JacobHayes merged commit d4093ad into golden Mar 20, 2023
@JacobHayes JacobHayes deleted the future-annotations branch March 20, 2023 05:51
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.

1 participant