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

Reload ns #80

Open
wants to merge 20 commits into
base: master
Choose a base branch
from
Open

Reload ns #80

wants to merge 20 commits into from

Conversation

barakl-madlan
Copy link

No description provided.

@fasfsfgs
Copy link
Contributor

Hi @barakl-madlan! Thanks a lot for your contribution!

Can I suggest you to format your code please?
We use default vscode typescript formatting (cmd + shift + f).
Thanks man!

@barakl-madlan
Copy link
Author

Sure, from now on I will format any file I am working on.
I planning to add more features when I will have some time.
Below are the features I have planned, let me know if any of you already working on something similar?

  1. I need to add verification that eastwood is in the dependencies otherwise disable the linter.
  2. Add test integration in the editor (running, seeing the result in the editor)
  3. running current block command
  4. copy namespace to clipboard command
  5. separate window (separate from the source control) to run commands in the current namespace

BR,

@fasfsfgs
Copy link
Contributor

Hi @barakl-madlan!

Thanks for showing interest in doing all of the above. =)

And thanks for formatting it. Let us know when you finish it.

Not sure about the other contributors but I'd say go for which one you find more interesting.
One thing I'd do is separate them in different PRs and keep them coming. =)

I was thinking about doing the running current block command but I didn't start it yet and I'm not sure when I'll have time to do it. So feel free to do it as well. I'll let you know if I start it.

@avli avli force-pushed the master branch 2 times, most recently from 7e1c85a to 068abc8 Compare November 8, 2022 15:35
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