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

remove file path syntax for imports #16

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

Conversation

austinkline
Copy link
Contributor

@austinkline austinkline commented Apr 9, 2023

Removes the file path imports since new versions of the flow cli don't need it. If we do this, you can use submodules to pull in standard repos so you don't have to copy other contracts anymore!

@austinkline austinkline requested review from bluesign and bjartek April 18, 2023 14:32
@bjartek
Copy link
Contributor

bjartek commented Apr 18, 2023

How does this work with testing frameworks?

When I tried this the last time i had to have an emulator running in project to get my editor to work. I dont like that.

Copy link
Contributor

@bjartek bjartek left a comment

Choose a reason for hiding this comment

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

Not sure if the tools are ready for this yet. Can see if i have time to get overflow to play with it

Co-authored-by: Bjarte S. Karlsen <bjarte@bjartek.org>
@austinkline
Copy link
Contributor Author

Sure enough, we need flow-js-testing to support this so it's blocked for now, we can circle back once that's been resolved

@turbolent
Copy link
Contributor

Looks like this has already been done now, with the switch to the Cadence testing framework?

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