Skip to content
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.

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

Add linq support for Select, Where, OnConflict, Columns, Order, Update, Set, and Delete #60

Merged
merged 8 commits into from
Jan 16, 2023

Conversation

acupofjose
Copy link
Contributor

@acupofjose acupofjose commented Jan 14, 2023

In reference to #58

  • Linq usage of Select(x => new[] { x.Name, x.CreatedAt })
  • Linq usage of Where(x => x.Id == 5)
  • Linq usage of Where(x => x.Name == "Top Gun" && x.CreatedAt <= new DateTime(2022, 08, 21))
    • Support usage of Contains operator:
      • String.Contains
      • ICollection.Contains
  • Support for chaining Where calls.
  • Linq for OnConflict
  • Linq for Columns
  • Linq for Order
  • Linq for Filter
  • Linq for Update and Set
  • Linq for Delete
  • Unit tests
  • Update README

@acupofjose acupofjose marked this pull request as draft January 14, 2023 16:13
@acupofjose acupofjose changed the title Add linq support for Select and Where Add linq support for Select, Where, OnConflict, Columns, and Order Jan 14, 2023
@acupofjose acupofjose marked this pull request as ready for review January 14, 2023 21:31
@acupofjose
Copy link
Contributor Author

@veleek any chance you'd be up for some more code review? I'd love your opinion on this!

@acupofjose acupofjose changed the title Add linq support for Select, Where, OnConflict, Columns, and Order Add linq support for Select, Where, OnConflict, Columns, Order, Update, Set, and Delete Jan 16, 2023
@acupofjose acupofjose merged commit c3c39b3 into master Jan 16, 2023
@acupofjose acupofjose deleted the dev/linq branch January 16, 2023 13:22
@veleek
Copy link

veleek commented Jan 16, 2023

Shoot, looks like I got to this too late. :)

@veleek
Copy link

veleek commented Jan 16, 2023

Took a quick glance through and didn't see anything immediately troubling. But I'm not super familiar with expressions either.

@acupofjose
Copy link
Contributor Author

Lol that’s what I get for coding on the weekend. Appreciate your time as always! Thanks @veleek

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