Skip to content

Conversation

@guneysus
Copy link
Contributor

  • Functions definitions default *args, **kwargs
  • Prop now shorthand of property
  • . snippet disabled, because overlaps autocomplete plugins.

@lpil
Copy link
Contributor

lpil commented Dec 14, 2014

Could you remove or rename the self snippet, rather than commenting it out please?

I'll leave someone else to too over the rest- I'm not a Python user, and I'm sure others have strong opinions on these snippets. :)

@guneysus
Copy link
Contributor Author

Now can be merged. Only *args, **kwargs improvements.

Copy link
Collaborator

Choose a reason for hiding this comment

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

_args, *_kwargs seems like an addition that people will basically always delete. I checked UltiSnips and we have ~3% of functions using one of them and none using both. Remove again?

@guneysus
Copy link
Contributor Author

Nothing changed in this commit except *args, **kwargs. The prop shorthand can be ignored.
I find difficult that typing *args, **kwargs. Now can be closed 👍

@guneysus guneysus closed this Dec 18, 2014
@lpil
Copy link
Contributor

lpil commented Dec 18, 2014

Why not add a snippet for *args, **kwargs? :)

@guneysus
Copy link
Contributor Author

Make sense. Ok. 👍

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.

3 participants