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.

Sign up for GitHub

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

Jump to bottom

Feat(create robo) no credential feature #253

Merged
merged 13 commits into from
Jun 4, 2024
Merged

Feat(create robo) no credential feature #253

merged 13 commits into from
Jun 4, 2024

Conversation

Nazeofel
Copy link
Contributor

@Nazeofel Nazeofel commented Jun 3, 2024

Huhh work ?

Nazeofel and others added 10 commits May 30, 2024 15:08
@Nazeofel
index.ts
4cc42d3 
Added -nc 'no credentials' feature to skip asking for credential, I also modified -p options so if the first option is '' it will skip asking for plugins. this update aims at easier testing as well as faster creation of templates.
@Nazeofel
update
62873cf
@Nazeofel
chore(index.ts): negated if to replace empty scope
745326a
@Nazeofel
oopsy it was not workey
4039563
@Nazeofel
changeset
5e78260
@Nazeofel
fix(index.ts)
a5e58da 
Condition to ask for no creds was broken
@Nazeofel
Merge branch 'main' into feat(create-robo)-no-credential-feature-
a1513f1
@Nazeofel
fix(index.ts)
b3ae1a1 
wasnt following best practices
@Nazeofel
Merge branch 'feat(create-robo)-no-credential-feature-' of https://gi…
07c7989 
…thub.com/Wave-Play/robo.js into feat(create-robo)-no-credential-feature-
@Nazeofel
fix
75a9e40 
???
@vercel Vercel
Copy link

vercel bot commented Jun 3, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
robo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 3, 2024 4:28pm

@Nazeofel Nazeofel closed this Jun 3, 2024
@Wave-Play Wave-Play locked and limited conversation to collaborators Jun 3, 2024
@Nazeofel
fix(index.ts)
834e338 
I hate commander
@Nazeofel
WIP
f37fc2c
@Pkmmte Pkmmte reopened this Jun 4, 2024
Pkmmte
Pkmmte approved these changes Jun 4, 2024
View reviewed changes
Copy link
Member

@Pkmmte Pkmmte left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@Pkmmte Pkmmte merged commit 8a948b1 into main Jun 4, 2024
2 of 3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Pkmmte Pkmmte Pkmmte approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@Nazeofel @Pkmmte