Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Refactor configuration tests #150

Merged
merged 3 commits into from
Aug 16, 2024
Merged

Refactor configuration tests #150

merged 3 commits into from
Aug 16, 2024

Conversation

teoljungberg
Copy link
Owner

@teoljungberg teoljungberg commented Aug 16, 2024

  • Add explicit return value of Fx.load. Returning true over
    ActiveRecord::SchemaDumper feels more expected.
  • Test Fx's configuration per file.

Returning true over ActiveRecord::SchemaDumper feels more expected
@teoljungberg teoljungberg changed the title Add explicit return value to Fx.load Refactor configuration tests Aug 16, 2024
@teoljungberg teoljungberg merged commit 6aed159 into master Aug 16, 2024
16 checks passed
@teoljungberg teoljungberg deleted the fx-load-true branch August 16, 2024 08:49
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant