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

Load :repo opt from app config #22

Closed
wants to merge 1 commit into from

Conversation

DanubioLima
Copy link
Contributor

@DanubioLima DanubioLima commented Jan 29, 2024

The repo option for SwissSchema is now loaded from config.exs If not found, it get the option from opts parameter, like before.

See #18

@joeljuca
Copy link
Owner

Hi @DanubioLima,

I've updated the issue with additional instructions for how it can be implemented (see #18). It was a bit loose, with some details left pending so I decided to write it down for us.

lib/swiss_schema.ex Show resolved Hide resolved
test/swiss_schema_test.exs Show resolved Hide resolved
test/swiss_schema_test.exs Show resolved Hide resolved
test/swiss_schema_test.exs Show resolved Hide resolved
test/swiss_schema_test.exs Show resolved Hide resolved
@joeljuca joeljuca changed the title Load repo from app default configs Load :repo opt from app config May 6, 2024
@joeljuca
Copy link
Owner

Continues at #24.

@joeljuca joeljuca closed this May 18, 2024
@DanubioLima DanubioLima deleted the feat/default-config branch May 21, 2024 02:07
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.

None yet

2 participants