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

Making "wing it" watcher ignore website folders #3956

Closed
tsuf239 opened this issue Aug 24, 2023 · 4 comments
Closed

Making "wing it" watcher ignore website folders #3956

tsuf239 opened this issue Aug 24, 2023 · 4 comments
Labels
⌨️ cli CLI ✨ enhancement New feature or request needs-discussion Further discussion is needed prior to impl 🎨 sdk SDK

Comments

@tsuf239
Copy link
Collaborator

tsuf239 commented Aug 24, 2023

Feature Spec

Assuming we have a folder containing our client-side application in one subfolder, and the wing file in another/at the root,
we want to prevent the wing watcher from watching the client-side files.
one way to do it is "knowing" what files to ignore (detect the website paths), and another, is "knowing" what files to include- only the wing file and its imports.

Use Cases

Real life wing app

Implementation Notes

No response

Component

Compiler, Wing Console, Development Environment

Community Notes

  • Please vote by adding a 👍 reaction to the issue to help us prioritize.
  • If you are interested to work on this issue, please leave a comment.
@tsuf239 tsuf239 added the ✨ enhancement New feature or request label Aug 24, 2023
@skyrpex
Copy link
Contributor

skyrpex commented Aug 29, 2023

This shouldn't be necessary if #3730 gets implemented, right?

@tsuf239
Copy link
Collaborator Author

tsuf239 commented Aug 29, 2023

This shouldn't be necessary if #3730 gets implemented, right?

right!

@garysassano
Copy link
Collaborator

So there's no need to complete this? It's not clear for me what is the "wing it's watcher". All I can see HERE is that it looks for a .w file, not for any subfolders.

@staycoolcall911 staycoolcall911 added the needs-discussion Further discussion is needed prior to impl label Oct 4, 2023
@tsuf239
Copy link
Collaborator Author

tsuf239 commented Oct 8, 2023

closing as a duplication (specific case) of #3730

@tsuf239 tsuf239 closed this as completed Oct 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⌨️ cli CLI ✨ enhancement New feature or request needs-discussion Further discussion is needed prior to impl 🎨 sdk SDK
Projects
Archived in project
Development

No branches or pull requests

4 participants