Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
    Vlad Faust
    @vladfaust
    And remember, when you're doing a PR, you create obligations for the maintainer to maintain your code onwards. So make sure the code is simple as possible
    Gian D
    @fusillicode_twitter
    👍🙇‍♂️
    Joakim Repomaa
    @repomaa
    hey @vladfaust
    i thought it's maybe easier if i write you here instead of going back and forth on the github issue
    Vlad Faust
    @vladfaust
    Yeah, hi, Joakim
    Joakim Repomaa
    @repomaa
    i'm not quite sure what you mean with the equals operator
    Vlad Faust
    @vladfaust
    Well, in your specs you're testing very basic language functionality -- whether did the DB variable changed
    I propose to write actual transaction call instead, it could be a simple SELECT 1 wrapped in a tx
    Just to make sure it works as intended
    Joakim Repomaa
    @repomaa
    ah ok
    it wouldn't compile if it didn't but ok :D
    so i should add those to exec_spec and query_spec?
    Vlad Faust
    @vladfaust
    Thanks. I would approve the PR, and would wait for the Crystal to upgrade to 0.30 before releasing
    Joakim Repomaa
    @repomaa
    sure, no rush
    Vlad Faust
    @vladfaust
    As exec and query do the same repo calls, only one place per db to test tx is enough. To be smart, you could put PG tx test in exec, and SQLite in query =)
    Joakim Repomaa
    @repomaa
    i'm just trying to find alternatives to crecto which isn't really being maintained anymore but is by far the most versatile orm i've seen. one thing that is hindering me from switching to Onyx::SQL is the lack of transaction support.
    Vlad Faust
    @vladfaust
    It would be super cool if you wrapped some actual TRANSACTION code in the query_spec, though
    Yeah, let's stick with that. Take some portion from */query_spec, and wrap, say, the post update, in a transaction -- for both DBs
    That would be the best
    Joakim Repomaa
    @repomaa
    alright
    Vlad Faust
    @vladfaust
    We're working on a language inspired by Crystal, which, apart from solving fundamental code issues Crystal has, would also aim for better ecosystem support. We hope this would solve the "not being maintained anymore" problem
    Joakim Repomaa
    @repomaa
    hm
    not sure a new language would fix that
    Vlad Faust
    @vladfaust
    would also aim for better ecosystem support
    Joakim Repomaa
    @repomaa
    that's something that can be fixed in crystal imho
    Vlad Faust
    @vladfaust
    Indeed, but this should be made by Manas :)
    Joakim Repomaa
    @repomaa
    why?
    Vlad Faust
    @vladfaust
    And they don't have interest in such actions
    Well, because it's their language and their ecosystem
    Joakim Repomaa
    @repomaa
    it's open source.
    it's not their ecosystem
    it's github
    or git*
    Vlad Faust
    @vladfaust
    I'll leave you with that
    Just take a note that I don't invest time in Crystal implementation of the framework
    No new features; supporting it, though
    Joakim Repomaa
    @repomaa
    ah good to know. I'll look elsewhere then :)
    Vlad Faust
    @vladfaust
    What are the alternatives?
    Joakim Repomaa
    @repomaa
    lucky
    Vlad Faust
    @vladfaust
    Yes
    Also Athena
    With Granite
    Joakim Repomaa
    @repomaa
    it's a tad too opinionated for my taste
    Vlad Faust
    @vladfaust
    PHP-inspired and lots of annotations :)
    Joakim Repomaa
    @repomaa
    lucky?
    Vlad Faust
    @vladfaust
    Athena
    Joakim Repomaa
    @repomaa
    k
    haven't looked at it
    usually i'm not looking for a web framework anyway
    just an orm