Skip to content

vdk-core: adopt pluggy 1.3#2614

Merged
antoniivanov merged 2 commits intomainfrom
person/aivanov/vdk-core-pluggy
Aug 28, 2023
Merged

vdk-core: adopt pluggy 1.3#2614
antoniivanov merged 2 commits intomainfrom
person/aivanov/vdk-core-pluggy

Conversation

@antoniivanov
Copy link
Copy Markdown
Contributor

Plugy realease of 1.3 on 25th of August introduced a changing renaming pluggy._callers._Result to pluggy.Result.

It can be found in the list of changes between 1.2 and 1.3 : pytest-dev/pluggy@1.2.0...1.3.0

As we rely on that as a type for the Result object , it breaks vdk.

This changing is adopting the pluggy change

@antoniivanov antoniivanov changed the title vdk-core: adopt plugin 1.3 vdk-core: adopt pluggy 1.3 Aug 28, 2023
Plugy realease of 1.3 on 25th of August introduced a changing renaming
pluggy._callers._Result to pluggy.Result.

It can be found in the list of changes between 1.2 and 1.3 :
pytest-dev/pluggy@1.2.0...1.3.0

As we rely on that as a type for the Result object , it breaks vdk.

This changing is adopting the pluggy change
@antoniivanov antoniivanov force-pushed the person/aivanov/vdk-core-pluggy branch from 9cddcf1 to 02a0d30 Compare August 28, 2023 12:44
Plugy realease of 1.3 on 25th of August introduced a changing renaming
pluggy._callers._Result to pluggy.Result.

It can be found in the list of changes between 1.2 and 1.3 :
pytest-dev/pluggy@1.2.0...1.3.0

As we rely on that as a type for the Result object , it breaks vdk.

This changing is adopting the pluggy change
@antoniivanov antoniivanov force-pushed the person/aivanov/vdk-core-pluggy branch from c152444 to eebc470 Compare August 28, 2023 14:24
@antoniivanov antoniivanov enabled auto-merge (squash) August 28, 2023 14:38
@antoniivanov antoniivanov merged commit 21695a3 into main Aug 28, 2023
@antoniivanov antoniivanov deleted the person/aivanov/vdk-core-pluggy branch August 28, 2023 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants