Skip to content

feat: [#358] Implement Exists, Find, FirstOrFail, Select methods#928

Merged
hwbrzzl merged 7 commits intomasterfrom
bowen/#358-9
Mar 2, 2025
Merged

feat: [#358] Implement Exists, Find, FirstOrFail, Select methods#928
hwbrzzl merged 7 commits intomasterfrom
bowen/#358-9

Conversation

@hwbrzzl
Copy link
Contributor

@hwbrzzl hwbrzzl commented Mar 2, 2025

📑 Description

goravel/goravel#358

@coderabbitai summary

✅ Checks

  • Added test cases for my code

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.50.

Benchmark suite Current: 6f6d525 Previous: 84fded1 Ratio
BenchmarkFile_ReadWrite 297096 ns/op 2072 B/op 28 allocs/op 184761 ns/op 2072 B/op 28 allocs/op 1.61
BenchmarkFile_ReadWrite - ns/op 297096 ns/op 184761 ns/op 1.61

This comment was automatically generated by workflow using github-action-benchmark.

@codecov
Copy link

codecov bot commented Mar 2, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.82%. Comparing base (84fded1) to head (d59930b).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #928   +/-   ##
=======================================
  Coverage   68.82%   68.82%           
=======================================
  Files         154      154           
  Lines       10200    10200           
=======================================
  Hits         7020     7020           
  Misses       2857     2857           
  Partials      323      323           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@hwbrzzl hwbrzzl changed the title feat: [#358] Implement Find method feat: [#358] Implement Exists, Find, FirstOrFail, Select method Mar 2, 2025
@hwbrzzl hwbrzzl changed the title feat: [#358] Implement Exists, Find, FirstOrFail, Select method feat: [#358] Implement Exists, Find, FirstOrFail, Select methods Mar 2, 2025
@hwbrzzl hwbrzzl marked this pull request as ready for review March 2, 2025 04:20
@hwbrzzl hwbrzzl requested a review from a team as a code owner March 2, 2025 04:20
@hwbrzzl hwbrzzl merged commit 61702e0 into master Mar 2, 2025
14 checks passed
@hwbrzzl hwbrzzl deleted the bowen/#358-9 branch March 2, 2025 04:24
almas-x pushed a commit that referenced this pull request Mar 12, 2025
* feat: [#358] Implement Find method

* chore: update mocks

* add Select method

* add Exists method

* update mock

* implement FirstOrFail

* update between

---------

Co-authored-by: hwbrzzl <24771476+hwbrzzl@users.noreply.github.com>
almas-x pushed a commit that referenced this pull request Mar 18, 2025
* feat: [#358] Implement Find method

* chore: update mocks

* add Select method

* add Exists method

* update mock

* implement FirstOrFail

* update between

---------

Co-authored-by: hwbrzzl <24771476+hwbrzzl@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant