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

Suggestion: replace .v.py with a custom viper extension #455

Closed
federicobond opened this issue Nov 10, 2017 · 8 comments
Closed

Suggestion: replace .v.py with a custom viper extension #455

federicobond opened this issue Nov 10, 2017 · 8 comments

Comments

@federicobond
Copy link

Relying on the .py extension may bring problems down the road with both people and tools confusing Viper source code with Python. Possible alternatives: .viper, .vi

Cute Animal Picture

article-2418451-1bc5dd92000005dc-964_634x421

That's a dog with a cute hair extension.

@fubuloubu
Copy link
Member

We had .vy in mind. However, someone needs to do all the work of creating GitHub syntax highlighters, vim plugins, etc etc with our language. If you want to be that person, be my guest.

The language is still growing and changing however, so I think the idea was to create a formal syntax spec a little later on to base all of these things on. I welcome any work someone is willing to do to figure out what would be required when we reach that point though.

@federicobond
Copy link
Author

Good, thanks for taking the time to respond @fubuloubu, I really appreciate it. For the moment I am more focused on Solidity, and some of these suggestions come from the lessons I learned along the way. It becomes much more difficult to change things once the language starts getting traction.

@jacqueswww
Copy link
Contributor

@federicobond Very valid point, I have been really fond of the .v.py because it just highlights immediately - we will definitely reach the point to adopt .vy and I'll help with syntax highlighting templates where I can.
@fubuloubu So it's looking up for using .vy looking at it's uniqueness: https://github.com/search?utf8=%E2%9C%93&q=extension%3Avy+NOT+nothack&type=Code

@fubuloubu
Copy link
Member

Is there a way to register that extension with GitHub? Perhaps we should do that now

@jacqueswww
Copy link
Contributor

This is the guide: https://github.com/github/linguist/blob/master/CONTRIBUTING.md I found. I see solidity's PR is still pending: github-linguist/linguist#3755

@fubuloubu
Copy link
Member

Dang, the requirement to have 1000s of usage examples will definitely require a lot of promoting in Beta phase. Totally makes sense from GitHub's standpoint, but it looks like we can't do anything now to get Viper in there other than to keep doing what we're doing and making a great language.

I think with the potential name change issue we have as well as not being in Beta yet, .v.py works for now.

@chgue
Copy link
Contributor

chgue commented Nov 15, 2017

Out of curiosity I created a bare-bones vim plugin for .vy files. Should cover most syntax cases but I haven't tested it thoroughly. The extension can be changed easily, especially considering the name change issue @fubuloubu mentioned.

@jacqueswww
Copy link
Contributor

@chgue are you still interested in keeping that vim plugin maintained ?

I have started updating all the outdated things, like the name & keywords. And will gladly PR it over to your repo - if you feel like maintaining it. Otherwise I will run with my own copy.

hskang9 added a commit to hskang9/sharding that referenced this issue Jun 21, 2018
Recently, vyper extension has been set to ```.vy``` as beta version releases.
Additional information is described [here](vyperlang/vyper#455)
hskang9 added a commit to hskang9/sharding that referenced this issue Jun 21, 2018
Recently, vyper extension has been set to ```.vy``` as beta version releases.
Additional information is described [here](vyperlang/vyper#455)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants