Skip to content

Commit

Permalink
Fix typo
Browse files Browse the repository at this point in the history
  • Loading branch information
@char101
char101 committed Feb 26, 2023
1 parent f3015be commit 43ac21d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion yapf/pytree/subtype_assigner.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
"""

from yapf.ylib2to3 import pytree
from yapf.ylib2to3.pgen2 import token as gammar_token
from yapf.ylib2to3.pgen2 import token as grammar_token
from yapf.ylib2to3.pygram import python_symbols as syms

from yapf.pytree import pytree_utils
Expand Down

4 comments on commit 43ac21d

@Spitfire1900
Copy link

Choose a reason for hiding this comment

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

Any chance you can open up a PR against yapf for this fix? It would be great to help resolve google#1045

@char101
Copy link
Owner Author

Choose a reason for hiding this comment

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

Sorry I don't have time for that and I doubt it will be merged because (1) yapf itself doesn't look to be actively maintained (2) embedding lib2to3 is a structural change and I'm not sure if that is the direction the yapf maintainers want.

@Spitfire1900
Copy link

Choose a reason for hiding this comment

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

I've forked your fork. Is this something that its okay if I pursue opening a PR for rather than yourself?

@char101
Copy link
Owner Author

@char101 char101 commented on 43ac21d Apr 6, 2023

Choose a reason for hiding this comment

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

Sure, that'd be great.

Please sign in to comment.