-
Notifications
You must be signed in to change notification settings - Fork 230
chore: add vscode config directory #2565
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
base: dev
Are you sure you want to change the base?
Conversation
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Visit the preview URL for this PR (updated for commit 9cd7730): https://walletrc--pull-2565-merge-59it7ek6.web.app (expires Tue, 18 Mar 2025 16:43:42 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: f66a4ff03faa546f12f0ae5a841bd9eff2714dcc |
Thanks - looks useful. Can you please add some info to the repo docs detailing the how/why of the |
There are some changes I'd like to make to this, which I will get to after I'm up-and-running. I'm marking this as a draft for the meantime so that it isn't merged while I'm away and to indicate that it's not ready to be reviewed. |
Simplifies the following use-cases:
Includes the
--web-port=
flag to simplify testing (login rather than register/restore)