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 Emulator Connection #467

Closed
jribbink opened this issue Nov 3, 2023 · 0 comments · Fixed by #466
Closed

Remove Emulator Connection #467

jribbink opened this issue Nov 3, 2023 · 0 comments · Fixed by #466
Assignees

Comments

@jribbink
Copy link
Contributor

jribbink commented Nov 3, 2023

The emulator connection has been a frequent pain point in the VSCode extension and it's time that we rip it out for a number of reasons:

  • It degrades user experience by introducing a number of configuration hurdles and bugs. The extension is no longer a plug-and-play Cadence tool, it's a fully-fledged flow client. It should be a Cadence extension for helping developers write Cadence better
  • The features provided were not particularly useful. Flow client capabilities belong in a tool like the CLI, and generally speaking most interactions with the chain will be either scripted (i.e. tests) or at the application level (i.e. FCL/dev wallet)
  • It doesn't do what it's supposed to well. Transaction/script arguments are clunky, difficult to use. Account management is confusing. It doesn't play nice when the emulator is connected to other tools, the list goes on.
  • It doesn't encourage a good development process. We should instead be pushing people into the Cadence Testing Framework to test their expected states, not running scripts ad-hoc and visually inspecting the result
  • It meant parsing flow.json in javascript. This was a recipe for disaster as far as bugs go.

The list goes on. There's really no compelling reason to have these features and they are the cause of 99% of the emulator issues seen today.

@jribbink jribbink self-assigned this Nov 3, 2023
@gregsantos gregsantos moved this to 🔖 Ready for Pickup in 🌊 Flow 4D Nov 6, 2023
@jribbink jribbink mentioned this issue Nov 7, 2023
6 tasks
@github-project-automation github-project-automation bot moved this from 🔖 Ready for Pickup to ✅ Done in 🌊 Flow 4D Nov 15, 2023
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 a pull request may close this issue.

1 participant