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

A couple missing rust commands #918

Merged
merged 2 commits into from
Jul 20, 2022
Merged

Conversation

RyanRuane
Copy link
Contributor

No description provided.

Copy link
Collaborator

@rntz rntz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks okay. What's the purpose of changing from typing to collections.abc though?

@@ -1,8 +1,9 @@
import itertools
import re
from collections import defaultdict
from collections.abc import Mapping
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's up with these changes from typing to collections.abc?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good question. I think my branch may have been out of sync and not up to date. I can put it back.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Our autoformatter started doing that last week. Apparently as of 3.9 they're starting to deprecate the versions from typing in favour of those from collections.abc

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I think this is coming from a new pyupgrade release.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this change was made in #921, so if you rebase this change should disappear.

@@ -48,6 +48,7 @@ toggle use: user.code_toggle_libraries()
## Simple aliases
borrow: "&"
borrow mutable: "&mut "
state (async | asynchronous): "async "
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I use "a sink" for this, as I found recognition better. Not sure our stance regarding these kind of games to improve recognition tho. cc/ @knausj85 @lunixbochs

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At some point it might be worth adding this under a common tag to make this consistent between languages.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fwiw we support this as a keyword in Javascript

@RyanRuane
Copy link
Contributor Author

rebased

@rntz rntz merged commit e74c7b6 into talonhub:main Jul 20, 2022
georgejdli pushed a commit to georgejdli/georgejdli_talon that referenced this pull request Jul 22, 2022
* A couple missing rust commands

* Added "a sink" to improve accuracy
nickvisut pushed a commit to nickvisut/knausj_talon that referenced this pull request Nov 1, 2022
* A couple missing rust commands

* Added "a sink" to improve accuracy
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.

4 participants