create-yorkie-app

[blurfx]

What this PR does / why we need it?

Pulled the create-yorkie-app branch that @se030 worked, to fix ci failing issue.

Any background context you want to provide?

See #643

What are the relevant tickets?

Checklist

• Added relevant tests or not required
• Didn't break anything

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (e11047d) 69.10% compared to head (0494fec) 69.12%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #690      +/-   ##
==========================================
+ Coverage   69.10%   69.12%   +0.02%     
==========================================
  Files          58       58              
  Lines        8787     8787              
  Branches      800      800              
==========================================
+ Hits         6072     6074       +2     
+ Misses       2455     2454       -1     
+ Partials      260      259       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[blurfx]

I think a problem has been fixed on 953c372.

ref: vercel/turborepo#4183

[hackerwins]

@se030 I have temporarily granted permission to push to this branch until this PR is merged.

@blurfx, @se030 I have two comments:

A. If the JS SDK examples are directly included in the create-yorkie-app release, we should consider it has dependencies on the JS SDK. If so, it would be good to use the same version as the JS SDK. Checking the version of create-yorkie-app can help determine which version of the JS SDK it refers to.

B. It seems that create-yorkie-app is a CLI. How do you plan to release it? If it is to be installed using npm -g create-yorkie-app, do we need to add a new package to npm?

[blurfx]

@hackerwins
A. Agree. currently, examples are included in the create-yorkie-app build, so I think it would be good to follow the same version. and if in the future create-yorkie-app can create an empty project too(boilerplate?), I think this case could be approached differently.

B. Yes, it should be deployed to npm.

[hackerwins]

Thanks for your contribution.

[hackerwins]

I published manually create-yorkie-app to npm. And then I could see the package below.
https://www.npmjs.com/package/create-yorkie-app

After that, I was able to install create-yorkie-app with npm install -g. But when I execute the CLI, I get the following error.

Did I miss anything?

[blurfx]

There seems to be a problem with the code in the overwriting the environment variable file.