Skip to content

SQLite support and some clean-up #3

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.

Sign up for GitHub

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

Jump to bottom
Open
waseigo wants to merge 22 commits into
base: main
Choose a base branch
from
Open

SQLite support and some clean-up #3

waseigo wants to merge 22 commits into from

Conversation

@waseigo
Copy link

@waseigo waseigo commented Sep 15, 2025

  • Works with SQLite3 (without row-level locking)
  • Conditionally disable locking in AccountBalance.lock_cond/2 when database is SQLite3
  • Test configuration for SQLite3
  • Tagged 2 locking-dependent tests that are disabled when database is SQLite3
  • Cleaned up multi-line doctests
  • Added documentation to key modules
  • Updated README.md for SQLite3 and locking
  • Misc. improvements according to Credo issues

@waseigo
add credo and optional SQLite3 dependencies
59f8bff
@waseigo
include migration scripts
f4175ba
@waseigo
ignore SQLite database files
a8cbd96
@waseigo
add test configuration for SQLite3
abf7d4c
@waseigo
add commented-out configuration for SQLite
b7eacaf
@waseigo
replace fetch_env/2 with compile_env/2, fix multi-line doctests, …
b7b878d 
…escape tuples vs. IAL
@waseigo
switch single-clause with to case
61dab88
@waseigo
add :sqlite3 to case block
97a0048
@waseigo
@moduledoc false (implementation details)
3f243a1
@waseigo
add @moduledoc
ff5161f
@waseigo
switch to compile_env/2, add @moduledoc, disable locking for SQLite3
c71e7f2
@waseigo
add @moduledoc, fix multi-line doctests, fix constraint name with p…
41df9c2 
…refix for SQLite, add `inspect/1` calls to messages
@waseigo
determine JSON type with get_env/2 of :db; mix format
08a02cd
@waseigo
tag tests that require locking to exlude them from SQLite-based tests
4595722
@waseigo
exclude tests that require locking when using SQLite
f312522
@waseigo
add @moduledoc
2b0bab3
@waseigo
update README.md
966cbaf
@waseigo
disable Credo warning for large numbers and underscores
a98a138
@waseigo
@moduledoc false
7a6ca4e
@waseigo
Replace compile-time attributes with runtime functions
a309f47
@waseigo
Update account_balance.ex
dd36cef
@waseigo
Update line.ex
a4b6bb5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@waseigo