nhedger ยท GitHub

@nhedger

Copy link Copy Markdown

Contributor

Merged

clue

@clue clue left a comment

Copy link Copy Markdown

Member

Choose a reason for hiding this comment

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

@nhedger Thank you!

Can you apply/squash this minor suggestion? Once this is in, we should also apply the same changes to the 3.x and 2.x branches ๐Ÿ‘

@nhedger

@clue

clue

clue approved these changes Jun 20, 2022

@clue clue left a comment

Copy link Copy Markdown

Member

Choose a reason for hiding this comment

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

@nhedger Thanks for the quick update, changes LGTM! ๐Ÿ‘

WyriHaximus

@WyriHaximus

@nhedger

nhedger deleted the chore/remove-leading-dollar branch

June 24, 2022 09:57

@clue

Copy link Copy Markdown

Member

@nhedger Glad this has been merged! Can you also apply the same changes to the 3.x and 2.x branches? ๐Ÿ‘

This was referenced

Jun 28, 2022

Merged

Merged

@nhedger

Copy link Copy Markdown

Contributor Author

Read the original on github.com โ†—