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

Adding Editors section under Resources #3672

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

miohtama
Copy link
Contributor

What I did

Add Editors section

Add hints how to tune up your code editor for Vyper

  • vyper-lsp
  • Remix

How I did it

Edit documentation

How to verify it

Check that Sphinx build deploys

Commit message

  • Vyper-friendly editor documentation added

Description for the changelog

  • Vyper-friendly editor documentation added

Cute Animal Picture


     ^__^   /                \
         (oo)  ( Milk is logical. )
  /-------\/ --'\________________/
 / |     ||
*  ||W---||
   ^^    ^^
Mr Spock's cow

@codecov-commenter
Copy link

codecov-commenter commented Nov 16, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.61%. Comparing base (f0afaf0) to head (790da53).

Current head 790da53 differs from pull request most recent head 5c8aebe

Please upload reports for the commit 5c8aebe to get more accurate results.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3672      +/-   ##
==========================================
- Coverage   91.12%   89.61%   -1.51%     
==========================================
  Files         106       80      -26     
  Lines       15308    11343    -3965     
  Branches     3365     2553     -812     
==========================================
- Hits        13949    10165    -3784     
+ Misses        925      791     -134     
+ Partials      434      387      -47     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@@ -15,6 +15,12 @@ General
- `Vyper greatest hits smart contract examples <https://github.com/pynchmeister/vyper-greatest-hits/tree/main/contracts>`_
- `A curated list of Vyper resources, libraries, tools, and more <https://github.com/stars/pcaversaccio/lists/vyper>`_

Editors
-------

Copy link
Member

Choose a reason for hiding this comment

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

maybe add the VScode plugin as well as the Vim plugins and such

Copy link
Contributor Author

Choose a reason for hiding this comment

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

vyper-lsp is both VS Code and Vim, or is there some other plugin as well?

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

yes to vim-vyper

for vscode vyper, tintinweb's version is slowly becoming unmaintained(?) but we could put in https://github.com/z80dev/vscode-vyper-lsp-extension

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