-
-
Notifications
You must be signed in to change notification settings - Fork 652
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
Update environments #2339
Update environments #2339
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
50f24c0
to
0639a62
Compare
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.
This is a biggie but I think this looks good
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.
@nunogois This is nice, but I was tempted to reject based on the size of the PR. Please try to break up the PR size into chunks in the future. It's much easier to review, and we lower the risk of introducing bugs because the reviewer can more easily reason about the code and take in the entire context of the PR. It was difficult for me to do that here.
https://linear.app/unleash/issue/2-357/update-environments-pages