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

Svnsairam setup turtlestack cli #5846

Closed

Conversation

svnsairam
Copy link
Contributor

Description

Scenarios Tested

Sample Commands

@svnsairam svnsairam requested review from taeold and blidd-google May 16, 2023 15:31
@TheIronDev
Copy link
Contributor

I believe you're going to want to feature-gate this.

@codecov-commenter
Copy link

codecov-commenter commented May 19, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (dc08ce4) 54.94% compared to head (96ca08c) 54.94%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #5846   +/-   ##
=======================================
  Coverage   54.94%   54.94%           
=======================================
  Files         339      339           
  Lines       23151    23151           
  Branches     4733     4733           
=======================================
  Hits        12721    12721           
  Misses       9303     9303           
  Partials     1127     1127           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

const projectId = setup?.rcfile?.projects?.default;
if (projectId) {
await requirePermissions({ ...options, project: projectId });
await Promise.all([ensure(projectId, "firebaseextensions.googleapis.com", "unused", true)]);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add a TODO here to replace with real API later?

@jamesdaniels jamesdaniels added the turtles I like Turtles label May 23, 2023
src/commands/init.ts Show resolved Hide resolved
src/commands/init.ts Show resolved Hide resolved
src/init/features/frameworkstack/index.ts Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
turtles I like Turtles
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants