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

watch-template is watching output folder #409

Closed
GordeevArt opened this issue Sep 11, 2020 · 14 comments
Closed

watch-template is watching output folder #409

GordeevArt opened this issue Sep 11, 2020 · 14 comments
Labels
bug Something isn't working

Comments

@GordeevArt
Copy link

GordeevArt commented Sep 11, 2020

Describe the bug

Discussed here asyncapi/html-template#61 - I remove unused files by hook and it coping files infinitely.

How to Reproduce

In https://github.com/asyncapi/html-template do
ag test/spec/asyncapi.yml ./ -o test/output --force-write --watch-template -p singleFile=true
than change some files.

image

Expected behavior

Not to generate files when output folder changes.

@GordeevArt GordeevArt added the bug Something isn't working label Sep 11, 2020
@github-actions
Copy link
Contributor

Welcome to AsyncAPI. Thanks a lot for reporting your first issue.

Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.

@jonaslagoni
Copy link
Member

jonaslagoni commented Sep 11, 2020

#337 should have fixed this issue 🤔 @GordeevArt what generator version are you using?
@derberg any thoughts on the issue? (Edit: just read the discussion nwm 😄)

@GordeevArt
Copy link
Author

@jonaslagoni I use latest @asyncapi/generator@1.0.0-rc.10 version

@derberg
Copy link
Member

derberg commented Sep 11, 2020

@jonaslagoni I forgot I fixed it some time ago 😆 I just checked quickly on local and for me my fix still works 😄

@GordeevArt are you a windows user? or?

@GordeevArt
Copy link
Author

@derberg Fedora Linux 5.7.15-100.fc31.x86_64

Try to change something in output folder (index.html).

it matters just for developing with parameter singleFile=true.

@derberg
Copy link
Member

derberg commented Sep 11, 2020

@GordeevArt I tried to do changes manually in output, also added a hook that you added without any parameters so it always runs, and no effect

I recorded this to show how I do it

@GordeevArt
Copy link
Author

@derberg ok, thanks for research. I do the same but have the bug. It is no critical.

@derberg
Copy link
Member

derberg commented Sep 11, 2020

@GordeevArt this is what we are using under the hood to watch for changes. Would be great if you could try out their CLI with --ignore flag, to see if this issue is caused by the chokidar library or we just do something wrong in the Generator

@GordeevArt
Copy link
Author

@derberg this still don't work for me

@derberg
Copy link
Member

derberg commented Sep 11, 2020

@GordeevArt you mean you tried https://github.com/kimmobrunfeldt/chokidar-cli indepedently from the generator and it was also detecting changes in ignored paths?

@github-actions
Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity 😴
It will be closed in 30 days if no further activity occurs. To unstale this issue, add a comment with detailed explanation.
Thank you for your contributions ❤️

@github-actions github-actions bot added the stale label Nov 11, 2020
@derberg
Copy link
Member

derberg commented Nov 11, 2020

@GordeevArt hey man, is it still valid?

@GordeevArt
Copy link
Author

Yes, but I don't use it now.

@derberg derberg removed the stale label Jul 5, 2023
@derberg
Copy link
Member

derberg commented Jul 5, 2023

if somebody experience similar issue, please instead use https://github.com/asyncapi/cli and any CLI issues report there 🙏🏼

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants