clasp icon indicating copy to clipboard operation
clasp copied to clipboard

Pushing V8 code into non-V8 project

Open bjohas opened this issue 5 years ago • 11 comments

When pushing V8 code into non-V8 project, the push fails. Howeer, the error message "Error: Syntax error: Missing ; before statement. ", while obviously true, didn't immediately make me realise that this is with V8/non-V8 code.

Expected Behavior

A warning message would be helpful, but may not be possible.

Specifications

2.3.0

bjohas avatar Apr 28 '20 12:04 bjohas

Could you wait #778 merging then return to this again?

oshliaer avatar May 27 '20 17:05 oshliaer

@bjohas are you pushing plain .js or .ts?

If its .js then catching the error would be nice to have.

PopGoesTheWza avatar May 27 '20 18:05 PopGoesTheWza

Plain .js, i.e., trying to push "V8-plain-js" into a non-V8 project. I'll look out for the PR merge and then try again.

bjohas avatar May 27 '20 18:05 bjohas

#778 have a cleaner error handling architecture but there is probably not yet a user friendly catch for this case

PopGoesTheWza avatar May 27 '20 18:05 PopGoesTheWza

OK, sure - I'll test again once https://github.com/google/clasp/pull/778 has been merged.

bjohas avatar May 27 '20 20:05 bjohas

Thanks for working on clasp!

bjohas avatar May 27 '20 20:05 bjohas

Can you test with the #791 unofficial release?

npm uninstall -g @google/clasp && npm install -g forked-clasp
clasp --version
# should be 2.4.0 or more

PopGoesTheWza avatar Jul 15 '20 19:07 PopGoesTheWza

Thank you - will do and report back!

bjohas avatar Jul 15 '20 19:07 bjohas

@bjohas what is the status for this issue?

PopGoesTheWza avatar Mar 19 '21 21:03 PopGoesTheWza

Should be fixed in https://github.com/google/clasp/releases/tag/v2.3.1

PopGoesTheWza avatar May 12 '21 22:05 PopGoesTheWza

Thank you so much - I'm sorry I didn't get round to testing it, as the old code had gone away. I'd be happy for this to be closed. If I come across the error again, I can always come back to this?

bjohas avatar May 14 '21 19:05 bjohas