pharo-rdbms / Pharo-SQLite3

Community-owned official SQLite3 binding for Pharo
MIT License
22 stars 20 forks source link

use rowClass instead of collect #87

Closed vonbecmann closed 1 year ago

vonbecmann commented 1 year ago

fix: #86 add SQLite3Column>>#cid: and SQLite3Column>>#notnull: those accessor are use by the cursor to initialize the column

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.04% :warning:

Comparison is base (1592f72) 80.04% compared to head (0a08252) 80.00%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #87 +/- ## ========================================== - Coverage 80.04% 80.00% -0.04% ========================================== Files 108 108 Lines 4359 4356 -3 ========================================== - Hits 3489 3485 -4 - Misses 870 871 +1 ``` | [Files Changed](https://app.codecov.io/gh/pharo-rdbms/Pharo-SQLite3/pull/87?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pharo-rdbms) | Coverage Δ | | |---|---|---| | [src/SQLite3-Core/SQLite3Column.class.st](https://app.codecov.io/gh/pharo-rdbms/Pharo-SQLite3/pull/87?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pharo-rdbms#diff-c3JjL1NRTGl0ZTMtQ29yZS9TUUxpdGUzQ29sdW1uLmNsYXNzLnN0) | `92.15% <100.00%> (-7.85%)` | :arrow_down: | | [src/SQLite3-Core/SQLite3Table.class.st](https://app.codecov.io/gh/pharo-rdbms/Pharo-SQLite3/pull/87?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pharo-rdbms#diff-c3JjL1NRTGl0ZTMtQ29yZS9TUUxpdGUzVGFibGUuY2xhc3Muc3Q=) | `100.00% <100.00%> (ø)` | | ... and [2 files with indirect coverage changes](https://app.codecov.io/gh/pharo-rdbms/Pharo-SQLite3/pull/87/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pharo-rdbms)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.