Skip to content

Allow automatically named indices #124

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.

Sign up for GitHub

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

Jump to bottom
Closed
ludaavics opened this issue Nov 14, 2019 · 1 comment · Fixed by #158
Closed

Allow automatically named indices #124

ludaavics opened this issue Nov 14, 2019 · 1 comment · Fixed by #158
Labels
bug Something isn't working priority-normal topic-stubs

Comments

@ludaavics
Copy link

Hi,

mypy rejects Index(None, "column1", "column2") as it expects a string. However, we are allowed to pass None if we want to use the automated naming

Thanks!
@ludaavics ludaavics changed the title Allow automatically names index Allow automatically named indices Nov 14, 2019
@ilevkivskyi
Copy link
Contributor

Thanks for reporting! PRs welcome.

@jdkandersson jdkandersson mentioned this issue Jun 7, 2020
Merged
@ludaavics ludaavics mentioned this issue Jul 7, 2020
Closed
ilevkivskyi pushed a commit that referenced this issue Jan 8, 2021
@jdkandersson
Make name argument optional (#158)
7b890f8 
Closes #124
brabadu pushed a commit to brabadu/sqlalchemy-stubs that referenced this issue Jan 15, 2021
@jdkandersson
Make name argument optional (dropbox#158)
2bb7a55 
Closes dropbox#124
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working priority-normal topic-stubs
Projects
None yet
Milestone
No milestone
2 participants
@ludaavics @ilevkivskyi