-
Notifications
You must be signed in to change notification settings - Fork 76
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
fix colima deploy #800
fix colima deploy #800
Conversation
Codecov ReportBase: 86.98% // Head: 86.92% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #800 +/- ##
==========================================
- Coverage 86.98% 86.92% -0.07%
==========================================
Files 97 97
Lines 8301 8304 +3
==========================================
- Hits 7221 7218 -3
- Misses 636 640 +4
- Partials 444 446 +2
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
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.
LGTM, have left a nitpick comment
conn_port: 5432 | ||
conn_uri: postgres://username:password@example.db.example.com:5432/schema | ||
conn_extra: {} | ||
- conn_id: local_postgres |
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.
nitpick: could we please fix the tests that cause the update to these test files every time we run the tests? Are your changes in this PR to settings/settings_test.go
fixing them?
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.
yes if the yaml file is like this than the tests shouldn't update the file any more
Description
This PR should fix colima deploy
🎟 Issue(s)
astro deploy
with colima is not working #615🧪 Functional Testing
📸 Screenshots
📋 Checklist
make test
before taking out of draftmake lint
before taking out of draft