-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Add new apphosting:env:import commmand #9206
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: master
Are you sure you want to change the base?
Conversation
Approved by committee ages ago. This will expand to remote env by just adding a --backend flag and can also include a dialog if neither backend nor apphosting.yaml is found. Only controversial change is that I updated the discoverBackendRoot to also look for .env files. This seems to also be a pretty good indication of the root of a website and helps make sure that this command is always operating on the same directory.
Summary of ChangesHello @inlined, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request introduces a significant quality-of-life improvement for developers working with App Hosting by adding a new Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
This pull request introduces a new command apphosting:env:import
to facilitate importing environment variables from a .env
file into apphosting.yaml
. This includes logic for diffing environments, handling conflicts, and creating or updating secrets in Google Secret Manager. The discoverBackendRoot
function is also updated to recognize .env
files as an indicator of a project's root directory. The implementation is well-tested with new spec files for the command's logic. My review found a couple of issues in the new command's implementation that could prevent it from working as expected, particularly when using the --source
flag or when targeting local/emulator environments. I've provided suggestions to fix these.
/gemini review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
This pull request introduces a new command apphosting:env:import
for importing environment variables from a .env
file into apphosting.yaml
. The implementation includes logic for diffing environments, handling conflicts, creating secrets for sensitive variables, and granting necessary permissions. The changes also update discoverBackendRoot
to recognize .env
files. The code is well-structured and includes comprehensive tests. I've provided a few suggestions to improve efficiency and maintainability.
b3ecbe3
to
faac277
Compare
Approved by committee ages ago. This will expand to remote env by just adding a --backend flag and can also include a dialog if neither backend nor apphosting.yaml is found.
Only controversial change is that I updated the discoverBackendRoot to also look for .env files. This seems to also be a pretty good indication of the root of a website and helps make sure that this command is always operating on the same directory.