Skip to content

Support external_identifiers#27

Merged
nicklayb merged 4 commits intomainfrom
external_identifiers
Sep 15, 2024
Merged

Support external_identifiers#27
nicklayb merged 4 commits intomainfrom
external_identifiers

Conversation

@nicklayb
Copy link
Owner

No description provided.

@nicklayb nicklayb marked this pull request as ready for review September 14, 2024 02:37
@nicklayb nicklayb merged commit 0f3cdea into main Sep 15, 2024
@coveralls
Copy link

Pull Request Test Coverage Report for Build 476767393135276533d9812388475e502213caa0-PR-27

Details

  • 50 of 50 (100.0%) changed or added relevant lines in 4 files are covered.
  • 32 unchanged lines in 2 files lost coverage.
  • Overall coverage increased (+0.9%) to 65.427%

Files with Coverage Reduction New Missed Lines %
lib/loupe/ecto.ex 1 98.36%
src/loupe_parser.erl 31 82.62%
Totals Coverage Status
Change from base Build 26fc5ec656ed681efdebc5c033281bda9175db66: 0.9%
Covered Lines: 827
Relevant Lines: 1264

💛 - Coveralls

nicklayb added a commit that referenced this pull request Mar 19, 2025
* Initial work.

* Needed to escape the tokens

* Fix pipeline

* Add match only single amp or single pipe

* More work to support single amp and or

* Support external_identifiers (#27)

* Support external_identifiers

* Add support for variables in params

* Fix type

* Fix Credo

* Bump version

* Stream query (#29)

* Working on documentation

* Working on tests

* Add tests for default comparator

* Add test fixtures

* Bump action version

* Bump image

* Update credo

* Done testing and documentation

* Remove unused file

* [main] Revert to hardcoded version

* Finish work
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.

2 participants