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

rosalina combo #1328

Closed
wants to merge 4 commits into from
Closed

rosalina combo #1328

wants to merge 4 commits into from

Conversation

irowiki
Copy link
Contributor

@irowiki irowiki commented Jul 16, 2024

Added rosalina and safeb9sinstaller combos.

NTRboot too perhaps?

@lifehackerhansol
Copy link
Contributor

hang on, the file names aren't gonna work, it needs to be 3ds or it won't be parsed
cc @FrozenChen

@irowiki
Copy link
Contributor Author

irowiki commented Aug 3, 2024

hang on, the file names aren't gonna work, it needs to be 3ds or it won't be parsed cc @FrozenChen

Oh I missed that.

Does this work combo.b9s.3ds.md

Or is there another way to make a multi command (.combo b9s, .combo rosalina)

@lifehackerhansol
Copy link
Contributor

It's not really built to work that way, you'd have to use real python code

@irowiki
Copy link
Contributor Author

irowiki commented Aug 4, 2024

It's not really built to work that way, you'd have to use real python code

Ah so basically, would just need to be .comborosalina and .combob9s etc (which is what I started with, but thought I could get fancy lol)

@lifehackerhansol lifehackerhansol changed the title safeb9sinstaller combo rosalina combo Aug 4, 2024
@lifehackerhansol
Copy link
Contributor

I merged just b9scombo for the time being. I'm still not 100% sure if rosalinacombo is necessary. Will give it more thought.

@irowiki
Copy link
Contributor Author

irowiki commented Aug 4, 2024

I merged just b9scombo for the time being. I'm still not 100% sure if rosalinacombo is necessary. Will give it more thought.

Maybe it could be worded better but we have to type it out a lot which is why I thought of it.

@irowiki
Copy link
Contributor Author

irowiki commented Aug 4, 2024

I merged just b9scombo for the time being. I'm still not 100% sure if rosalinacombo is necessary. Will give it more thought.

We could also add that if the buttons (or rosalina in general) aren't working, ensure you are on the (latest luma) and download (this config) and link to the guide config.

At that point .rosalina would work as an alias too hmm

@irowiki
Copy link
Contributor Author

irowiki commented Aug 4, 2024

Oh wait, I did not realize .rosalinabutton was a thing, hasn't really been used all that much.

Could just kinda... merge this into that and add a few aliases.

irowiki added 2 commits August 4, 2024 14:15
Moved to rosalinabutton
Since there was an existing command, merged the rosalinacombo file into this one
@irowiki
Copy link
Contributor Author

irowiki commented Aug 4, 2024

@lifehackerhansol What do you think? .rosalina makes more sense to broadly cover the command, everything else added as an alias.

@irowiki irowiki closed this Aug 14, 2024
@irowiki irowiki deleted the patch-10 branch August 14, 2024 05:03
@lifehackerhansol
Copy link
Contributor

merged into 4694865

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants