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

fix: Display a better error message when database URI is null #8015

Conversation

edgarrmondragon
Copy link
Collaborator

@edgarrmondragon edgarrmondragon commented Aug 12, 2023

@netlify
Copy link

netlify bot commented Aug 12, 2023

Deploy Preview for meltano canceled.

Name Link
🔨 Latest commit 2c37843
🔍 Latest deploy log https://app.netlify.com/sites/meltano/deploys/64da5f9193b83500082bffc0

@edgarrmondragon edgarrmondragon force-pushed the 8014-bug-exception-thrown-due-to-missing-environment-variable-for-database_uri-could-be-much-clearer branch from 105d857 to 610474b Compare August 12, 2023 02:35
@codecov
Copy link

codecov bot commented Aug 12, 2023

Codecov Report

Merging #8015 (2c37843) into main (37fe05a) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #8015   +/-   ##
=======================================
  Coverage   89.53%   89.54%           
=======================================
  Files         297      297           
  Lines       21903    21921   +18     
  Branches     2443     2446    +3     
=======================================
+ Hits        19611    19629   +18     
  Misses       1936     1936           
  Partials      356      356           
Files Changed Coverage Δ
src/meltano/core/db.py 81.01% <100.00%> (+1.84%) ⬆️
tests/meltano/core/test_db_connection.py 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@edgarrmondragon edgarrmondragon marked this pull request as ready for review August 14, 2023 14:10
…ironment-variable-for-database_uri-could-be-much-clearer
…ironment-variable-for-database_uri-could-be-much-clearer
@edgarrmondragon edgarrmondragon merged commit 892ebca into main Aug 14, 2023
43 checks passed
@edgarrmondragon edgarrmondragon deleted the 8014-bug-exception-thrown-due-to-missing-environment-variable-for-database_uri-could-be-much-clearer branch August 14, 2023 19:48
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.

bug: Exception thrown due to missing environment variable for database_uri could be much clearer
2 participants