search

Zaid-Ajaj / LiteDB.FSharp

Advanced F# Support for LiteDB, an embedded NoSql database for .NET with type-safe query expression through F# quotations
MIT License
181 stars 20 forks source link

Add liteRepository & LiteQueryable extensions #6

Closed humhei closed 6 years ago

humhei commented 6 years ago

Should i merge it myself ? @Zaid-Ajaj Or if you want keep the right to review changes? 😏 Maybe i am still a new one to fsharp world and git world

Zaid-Ajaj commented 6 years ago

Feel free to merge if you are finished working on the PR :wink:

I did add a review to the code for updateItem, you can comment there or just here if that is easier for you

When PR is merged, I will publish ASAP

humhei commented 6 years ago

Maybe we should add a script to auto publish to nuget using ci when pr is merged ?

humhei commented 6 years ago

never mind, as for now only 5 prs existed

Zaid-Ajaj commented 6 years ago

Maybe we should add a script to auto publish to nuget using ci when pr is merged ?

Not every PR is a bug fix or a feature, you might want to just refactor code for better maintainability or even just update README.md. Also I might want to change stuff after a PR so manual publish is better(TM)

Zaid-Ajaj commented 6 years ago

Published 1.3.0 :smile: should be available soon on nuget