Skip to content

Conversation

@gpeacock
Copy link
Contributor

@gpeacock gpeacock commented Dec 3, 2024

Also removes file close warnings in tests.

@gpeacock gpeacock requested review from dyro and tmathern and removed request for tmathern December 3, 2024 19:26
Copy link
Collaborator

@tmathern tmathern left a comment

Choose a reason for hiding this comment

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

Not going to argue with a formatter

load_settings_file,
)
from .c2pa import Error, SigningAlg, CallbackSigner, sdk_version, version, load_settings
from .c2pa.c2pa import (
Copy link
Collaborator

Choose a reason for hiding this comment

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

I never saw the syntax with parenthesis before, hope it doesn't have adverse strange side-effects. After researching it, that's allowed only for Python 3.10 and later, so Python-wise should be fine.

}
}

// pub fn validation_state(&self) -> Result<ValidationState> {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is it debug code?

@tmathern
Copy link
Collaborator

Closing as this is now obsolete.

@tmathern tmathern closed this Jun 19, 2025
@tmathern tmathern deleted the gpeacock/formatting branch October 4, 2025 03:37
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.

3 participants