Skip to content

rename "method" to "use_array" and make it keyword only#2609

Merged
j9ac9k merged 1 commit intopyqtgraph:masterfrom
pijyoi:rename-kwarg
Feb 11, 2023
Merged

rename "method" to "use_array" and make it keyword only#2609
j9ac9k merged 1 commit intopyqtgraph:masterfrom
pijyoi:rename-kwarg

Conversation

@pijyoi
Copy link
Copy Markdown
Contributor

@pijyoi pijyoi commented Feb 10, 2023

Even though PrimitiveArray is not public facing API, its __init__ argument "method : bool" was not named correctly.

This rename was originally meant to be part of #2596, but as #2596 needs more testing, the rename is extracted out to get it in before the next release.

document explicitly however that PrimitiveArray is not public facing
API.
@j9ac9k
Copy link
Copy Markdown
Member

j9ac9k commented Feb 11, 2023

@pijyoi I really appreciate your work on getting these lower-level interfaces working!

@j9ac9k j9ac9k merged commit d1653ec into pyqtgraph:master Feb 11, 2023
@pijyoi pijyoi deleted the rename-kwarg branch February 11, 2023 04:45
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.

2 participants