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

Remove deprecated UnionAll Vararg #1859

Merged
merged 2 commits into from
Sep 21, 2024
Merged

Conversation

danielwe
Copy link
Contributor

Gets rid of a depwarn during precompilation

`...` is better understood by more users and easier on the eyes
@danielwe
Copy link
Contributor Author

To be clear, it's the change at line 1070 that fixes the depwarn, the others are just more or less related docstring improvements (most importantly making sure not to use deprecated syntax, but I also think ... reads better than Vararg in a docstring signature)

@wsmoses wsmoses merged commit a08b810 into EnzymeAD:main Sep 21, 2024
10 of 24 checks passed
@danielwe danielwe deleted the varargunionall branch October 7, 2024 03:44
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