ApeWorX / ape-vyper

Vyper compiler plugin for the Ape Framework, using VVM
https://www.apeworx.io/
Apache License 2.0
26 stars 9 forks source link

Plugin does not load #22

Closed mullahfaizal closed 3 years ago

mullahfaizal commented 3 years ago

Tried using Python 3.8 and 3.9, latest ape.

$ ape plugins add vyper
WARNING: $GITHUB_ACCESS_TOKEN not set, skipping 2nd class plugins
Install unknown 3rd party plugin 'ape_vyper'? [y/N]: y
INFO: Installing ape_vyper...
$ ape plugins list
WARNING: Error loading plugin package 'ape_vyper'
WARNING: $GITHUB_ACCESS_TOKEN not set, skipping 2nd class plugins
INFO: No plugins installed
$ ape compile
WARNING: Error loading plugin package 'ape_vyper'
WARNING: No compilers detected for the following extensions: .vy

I don't have this issue with solidity

fubuloubu commented 3 years ago

@unparalleled-js :thinking: I believe we need a way to expose what these errors are via logging in ape core so it's easier to track down what the issues are in the plugin

fubuloubu commented 3 years ago

@mullahfaizal note what is happening here is that the plugin is currently not compatible with ape core, and so loading the plugin is skipped, meaning that the compiler for .vy files are not available when you execute the ape compile command.

We will work on a fix for this plugin so that it is compatible with the latest ape core cc @sabotagebeats

sabotagebeats commented 3 years ago

@mullahfaizal note what is happening here is that the plugin is currently not compatible with ape core, and so loading the plugin is skipped, meaning that the compiler for .vy files are not available when you execute the ape compile command.

We will work on a fix for this plugin so that it is compatible with the latest ape core cc @sabotagebeats

@fubuloubu I believe this is a breaking change in Ape 0.1.0-alpha.24. Compilation of vyper files does work in the current docker apeworx distro for version Ape 0.1.0-alpha.23. cc @unparalleled-js

@mullahfaizal as a workaround please try using ape version 0.1.0-alpha.23 until this bug is resolved.

antazoey commented 3 years ago

Yep, ape core CLI tooling is currently moving around, changing.

antazoey commented 3 years ago

@unparalleled-js 🤔 I believe we need a way to expose what these errors are via logging in ape core so it's easier to track down what the issues are in the plugin

https://github.com/ApeWorX/ape/pull/175