Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Call replacement functions from deprecated functions. #205

Conversation

dlyongemallo
Copy link
Contributor

This may be a breaking change for users who are calling the deprecated functions. Presumably, though, this change actually produces the correct intended behaviour for these calls.

To force the old behaviour, there is an optional parameter force_deprecated_behavior (defaults to False) which can be set to True.

Fixes #203.

…lt to `True`, i.e., noop).

When set to `False`, the functions will call their replacements instead.
…deprecated functions so that they call their replacements.
@dlyongemallo
Copy link
Contributor Author

Is there a way to notify users of the library? It would be good to be able to communicate deprecation notices to users.

@jvdwetering jvdwetering merged commit e701d74 into zxcalc:master Apr 17, 2024
3 checks passed
@dlyongemallo dlyongemallo deleted the call_replacements_from_deprecated_functions branch April 17, 2024 18:23
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.

Graph.to_json followed by pyzx.json_to_graph produces an error
2 participants