Skip to content

Allow overriding default context w/ EDN or JSON file#78

Merged
cap10morgan merged 2 commits intomainfrom
10-23-Allow_overriding_default_context_w/_EDN_or_JSON_file
Oct 24, 2023
Merged

Allow overriding default context w/ EDN or JSON file#78
cap10morgan merged 2 commits intomainfrom
10-23-Allow_overriding_default_context_w/_EDN_or_JSON_file

Conversation

@cap10morgan
Copy link
Contributor

@cap10morgan cap10morgan commented Oct 23, 2023

Closes #76

This is less awkward than stuffing default contexts into an env var.

Copy link
Contributor Author

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

@cap10morgan cap10morgan requested a review from a team October 23, 2023 20:14
@cap10morgan cap10morgan force-pushed the 10-23-Allow_overriding_default_context_w/_EDN_or_JSON_file branch from 6670c69 to be3c98d Compare October 23, 2023 20:18
@cap10morgan cap10morgan force-pushed the 10-23-Update_to_latest_db_update_tests_accordingly branch 2 times, most recently from 3082b90 to 35352a5 Compare October 23, 2023 20:49
Copy link
Contributor

@zonotope zonotope left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

📃

Copy link
Contributor Author

cap10morgan commented Oct 24, 2023

Merge activity

Base automatically changed from 10-23-Update_to_latest_db_update_tests_accordingly to main October 24, 2023 16:00
@cap10morgan cap10morgan merged commit 4171b12 into main Oct 24, 2023
@cap10morgan cap10morgan deleted the 10-23-Allow_overriding_default_context_w/_EDN_or_JSON_file branch October 24, 2023 16:00
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.

Add an option to load default conn context from a file

2 participants