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

[Feature testing] clasp swallows or delays output to the terminal #28

Closed
oshliaer opened this issue Feb 19, 2021 · 3 comments
Closed

[Feature testing] clasp swallows or delays output to the terminal #28

oshliaer opened this issue Feb 19, 2021 · 3 comments

Comments

@oshliaer
Copy link

Intro

I'm from there google/clasp/issues/805.

My problem is that the clasp is swallowing the output or locking the output to the terminal.

I'm using spawn calls to run clasp under the webpack watch mode.

The main approach is

spawn('clasp', 'run', {
  stdio: 'inherit',
});

Two things happen there.

  1. сlasp pushes data. But it does not add output to console
  2. the console doesn't respond to Ctrl+C input. I need to call an event from webpack (it is called first in the current console) then I can press CTRL+C.

This rarely happens. I don't find a pattern.

The reasons for this topic

I'm going to test this fork for the same problem. I will respond within 2 weeks.

@oshliaer
Copy link
Author

While testing I did not find any problems with terminal delays. But I still don't understand why it is happening with origin clasp 🤷‍♂️.

@PopGoesTheWza
Copy link
Owner

@contributorpw original repo no longer has google maintainers. There is no one to release an updated @google/clasp npm package currently.

This forked repo contains all the refactored code I merged into the official repo (but can't release on npm) and a few additions.

I released and update a forked-clasp npm package to make the many fixes and new features available.

Since the last official google/clasp npm release, I did a massive refactor of the code, especially to cleanup pending promises and loose async function which are likely related to your original issue.

@oshliaer
Copy link
Author

Regarding the issue. This version does not have the problems described above.

I would like to stay on this version, because I like the extra functionality.

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

No branches or pull requests

2 participants