Skip to content

Conversation

@fitzgen
Copy link
Member

@fitzgen fitzgen commented Jul 13, 2020

Also update the set up and testing instructions.

Also update the set up and testing instructions.
CONTRIBUTING.md Outdated
So far this extension has been written by folks who are primarily Rust
programmers, so it's highly likely that there's some faux pas in terms of Python
idioms. Feel free to create a PR to help make things more idiomatic if you see
something!
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm pretty sure that by this point, this extension does qualify as "idiomatic Python"!

Copy link
Member Author

Choose a reason for hiding this comment

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

Heh yeah I'm happy to remove this clause if its unnecessary now (I'm also not really in a position to judge whether this package is idiomatic or not 😅 )

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm sure there are a few things that could be further improved, but Python packages aren't all that consistent in general. Wasmtime-py seems at the very least better than average to me, which means the warning no longer applies!

@alexcrichton
Copy link
Member

I've pushep up a few wording tweaks which makes the ask for help on idioms a bit less zealous (but still there because I'm paranoid), and I've also freshened up the instructions for handling the native lib.

@alexcrichton alexcrichton reopened this Jul 13, 2020
@alexcrichton alexcrichton merged commit 02a2af5 into bytecodealliance:main Jul 13, 2020
@fitzgen fitzgen deleted the contributing branch July 13, 2020 20:08
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