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

Package generator is broken for js option #156

Closed
reeveng opened this issue Jul 3, 2023 · 3 comments · Fixed by #157
Closed

Package generator is broken for js option #156

reeveng opened this issue Jul 3, 2023 · 3 comments · Fixed by #157
Assignees
Labels
squad:devops Issue to be handled by the Devops team. type:bug This issue reports a buggy (incorrect) behavior.
Milestone

Comments

@reeveng
Copy link

reeveng commented Jul 3, 2023

📝 Provide detailed reproduction steps (if any)

I tried to generate a package with --js as option, always results in an error after going through several phases.

Steps to reproduce:

  1. run npx ckeditor5-package-generator@latest @somerandomname/ckeditor5-some-random-name --use-yarn
  2. select js as language
  3. wait a bit for the package to start generating
  4. observe this error:
image

editorial note:

  • the apparent result seems to somewhat work when you try to run it 👍
  • also noticed that recently all or nearly all plugins switched to typescript, if javascript plugins are not intended, then remove the option?
  • it doesn't seem possible to add translated values to custom-plugins?

✔️ Expected result

Not sure, but maybe not an error?

❌ Actual result

An error that's confusing, did it work, did it not, it generated files, what am I supposed to do with them, yeet them and delete them, or treasure them ...?

❓ Possible solution

not sure man, haven't looked at the code of the package generator

If you'd like to see this fixed sooner, add a 👍 reaction to this post.

@reeveng reeveng added the type:bug This issue reports a buggy (incorrect) behavior. label Jul 3, 2023
@Witoso
Copy link
Member

Witoso commented Jul 3, 2023

Thanks for the report! I cannot reproduce it but I'm on MacOS. And it looks like the OP shows Windows. @pomek could your team take a look?

  • also noticed that recently all or nearly all plugins switched to typescript, if javascript plugins are not intended, then remove the option?

This doesn't matter, it determines what will be generated to your src dir, additional files needed, and different webpack configs.

  • it doesn't seem possible to add translated values to custom-plugins?

Not sure what you mean here, please elaborate :)

@Witoso Witoso added the squad:devops Issue to be handled by the Devops team. label Jul 3, 2023
@pomek
Copy link
Member

pomek commented Jul 4, 2023

Something is wrong when a directory contains a space. It can be reproduced on both path styles: Windows and Unix.

image

@pomek
Copy link
Member

pomek commented Jul 4, 2023

After removing the following lines, it works as expected.

@pomek pomek transferred this issue from ckeditor/ckeditor5 Jul 4, 2023
@CKEditorBot CKEditorBot added the status:planned Set automatically when an issue lands in the "Sprint backlog" column. We will be working on it soon. label Jul 4, 2023
@pomek pomek self-assigned this Jul 4, 2023
@CKEditorBot CKEditorBot added status:in-progress Set automatically when an issue lands in the "In progress" column. We are working on it. and removed status:planned Set automatically when an issue lands in the "Sprint backlog" column. We will be working on it soon. labels Jul 4, 2023
psmyrek added a commit that referenced this issue Jul 5, 2023
Fix (generator): Removed the current working directory from a package manager command when installing dependencies. A new process is already spawned in the directory. Hence, there is no need to duplicate the path. Thanks to that, a space in the path will not crash the generator while installing dependencies. Closes #156.
@CKEditorBot CKEditorBot removed the status:in-progress Set automatically when an issue lands in the "In progress" column. We are working on it. label Jul 5, 2023
@CKEditorBot CKEditorBot added this to the iteration 65 milestone Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
squad:devops Issue to be handled by the Devops team. type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants