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

Node module resolution #324

Merged
merged 17 commits into from
Dec 22, 2022
Merged

Node module resolution #324

merged 17 commits into from
Dec 22, 2022

Conversation

bohendo
Copy link
Contributor

@bohendo bohendo commented Dec 16, 2022

Fixes issues related to improper resolution of dependencies from node_modules:

fixes #279
fixes #258
fixes #321
fixes #305
fixes #290
fixes #255

scripts/ci_test_brownie.sh Outdated Show resolved Hide resolved
This reverts commit 7f7047b.
@montyly montyly changed the base branch from master to dev December 16, 2022 17:06
pyproject.toml Outdated
@@ -11,6 +11,7 @@ invalid-name,
fixme,
too-many-return-statements,
too-many-ancestors,
too-many-statements,
Copy link
Member

Choose a reason for hiding this comment

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

I am not sure about removing this warning. If pylint raises this its usually because a function would benefit from being split between internal functions

Remove working_dir usage to reduce complexity
@montyly montyly merged commit 567ffb0 into dev Dec 22, 2022
@montyly montyly deleted the node-module-resolution branch January 4, 2023 20:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment