#pyramid

/

      • otter768 joined the channel
      • stevepiercy_away is now known as stevepiercy
      • mejymejy joined the channel
      • joulez is now known as joules
      • blr_ has quit
      • blr joined the channel
      • mejymejy has quit
      • KageSenshi has quit
      • stevepiercy
        yah-veh-skript
      • Lcaracol joined the channel
      • stevepiercy is now known as stevepiercy_away
      • marplatense has quit
      • github joined the channel
      • github
        [pyramid_tm] mmerickel pushed 1 new commit to master: http://git.io/vBRqM
      • pyramid_tm/master f188c5b Michael Merickel: run docs only against python3
      • github has left the channel
      • LeoAtWork joined the channel
      • Method__ has quit
      • github joined the channel
      • [pyramid] mmerickel pushed 3 new commits to master: http://git.io/vBROX
      • pyramid/master a708d35 Michael Merickel: remove py2-docs from tox.ini and reorder so coverage is last
      • pyramid/master 0030fba Michael Merickel: add docs to travis builds
      • pyramid/master 2d9ed3f Michael Merickel: Merge pull request #2152 from mmerickel/fix/drop-py2-docs...
      • github has left the channel
      • github joined the channel
      • [pyramid] mmerickel pushed 2 new commits to 1.6-branch: http://git.io/vBR36
      • pyramid/1.6-branch 04d2ae7 Michael Merickel: remove py2-docs from tox.ini and reorder so coverage is last
      • pyramid/1.6-branch 7bff487 Michael Merickel: add docs to travis builds
      • github has left the channel
      • raydeo
        stevepiercy_away: couldn't help but notice that HACKING.txt is slightly different on 1.6-branch and master
      • dunno if it's something backports missed or whatever else, but just pointing it out fwiw
      • github joined the channel
      • github
        [pyramid] mmerickel pushed 1 new commit to 1.6-branch: http://git.io/vBRsj
      • pyramid/1.6-branch 739788a Michael Merickel: oops, drop pep8 from 1.6-branch
      • github has left the channel
      • LeoAtWork has quit
      • otter768 joined the channel
      • mejymejy joined the channel
      • GitHub38 joined the channel
      • GitHub38
        [pyramid_jinja2] mmerickel pushed 1 new commit to master: http://git.io/vBRci
      • pyramid_jinja2/master f978ec6 Michael Merickel: use tox that tests py2+py3 coverage
      • GitHub38 has left the channel
      • GitHub52 joined the channel
      • GitHub52
        [pyramid_jinja2] mmerickel pushed 1 new commit to master: http://git.io/vBRcS
      • pyramid_jinja2/master e55445c Michael Merickel: add full coverage and docs to travis
      • GitHub52 has left the channel
      • mejymejy has quit
      • ricoalpha joined the channel
      • ricoalpha
        hi, anyone using statsd + pyramid?
      • cguardia
        ricoalpha: substanced (a pyramid app server) uses statsd
      • ppaez joined the channel
      • Method__ joined the channel
      • skalfyfan joined the channel
      • raydeo
        so tentative plan for pyramid_tm is to refactor the tween into 2 tweens
      • inklesspen
        interesting
      • raydeo
        one retryable tween under excview and a new tween over excview that handles abort/commit
      • this way you can still use the transactions inside excviews
      • this also gives certain exception views the ability to override the exception and mark the transaction to be committed, possibly
      • anyway... just saying it out loud before I start playing with it
      • inklesspen
        oh cool
      • that makes sense
      • ricoalpha
        cguardia: what library did you use?
      • cguardia
        ricoalpha: statsd
      • ricoalpha
        and did you configure the settings through pyramid?
      • cguardia
        ricoalpha: no pyramid_metrics...you can see the integration code here: https://github.com/Pylons/substanced/blob/maste...
      • tiwula has quit
      • Lcaracol has quit
      • ninkotech has quit
      • ninkotech joined the channel
      • ricoalpha has quit
      • otter768 joined the channel
      • raydeo
      • d42 joined the channel
      • ricoalpha joined the channel
      • tiwula joined the channel
      • ricoalpha
        cool
      • would be perfect to have it as a seperate interface to include in pyramid
      • NotreDev joined the channel
      • tiwula has quit
      • zepolen has quit
      • ricoalpha has quit
      • github joined the channel
      • github
        [pyramid_tm] mmerickel pushed 1 new commit to master: http://git.io/vBRQH
      • pyramid_tm/master 8b0a1b5 Michael Merickel: remove py2.5 syntax
      • github has left the channel
      • sakal has quit
      • stevepiercy_away is now known as stevepiercy
      • zepolen joined the channel
      • stevepiercy
        raydeo ok, i'll have a look
      • x58
        raydeo: lol =)
      • raydeo
        oddly I wrote (wht I can tell) is a very sane integration test for pyramid_tm and it fails on 1.2 and 1.3
      • stevepiercy
        i would say that the guy in that comic strip is me, but he's probably wearing pants
      • x58
        Is 1.6-branch not pep8 BTW?
      • raydeo
        I see what you did there
      • x58: I thought it was but apparently not so meh
      • x58
        raydeo: Time to move on from 1.3 :P
      • raydeo
        x58: well I'm fairly certain these tests shouldn't fail on 1.3
      • x58
        Speaking of pants, got pizza coming so I am going to find some.
      • stevepiercy
        raydeo yeah, looks like master was not backported to 1.6 for HACKING.txt, as we banished submodules
      • x58
        No more version branching until we are ready to release...
      • stevepiercy
        i'll fix
      • ramteid joined the channel
      • raydeo
        no request methods in 1.2 sucks
      • x58
      • antonv6 is now known as av6
      • stevepiercy
        x58 raydeo this is a policy question for you: https://github.com/Pylons/pyramid/pull/2148
      • otter768 joined the channel
      • raydeo
        I'm not super excited about it
      • x58
        I'm a no on it.
      • raydeo
        considering adding it to that file usually means work for me... and I don't use dash
      • x58
        I liked Dash at one point, but then it became nasty shareware that didn't work right on the docsets I was already using.
      • raydeo
        RELEASING.txt is a one-man affiar generally
      • github joined the channel
      • github
        [pyramid] stevepiercy pushed 2 new commits to 1.6-branch: http://git.io/vBRNE
      • pyramid/1.6-branch 3bf245b Steve Piercy: backport master to 1.6-branch
      • pyramid/1.6-branch ab49af1 Steve Piercy: Merge pull request #2153 from stevepiercy/1.6-branch...
      • github has left the channel
      • stevepiercy
        ok, i wasn't super excited about it myself, i'll close the PR
      • raydeo
        ah fun... damn I don't like a lot of our tests
      • there's such an anti-integration testing stigma in our official projects
      • I feel like any time I get into them and look at the tests there's so much mocked out that it's hard to tell what's going on
      • like earlier this week when I got into pyramid_nacl_session and the tests weren't even using nacl's SecretBox
      • x58
        lol
      • raydeo
        or now in pyramid_tm where I write literally one integration tests that tries the tm on a real request and it fails immediately on pyramid 1.2/1.3, despite lots of tests testing the feature that fails because just too much is mocked out
      • x58
        Remember that fix I was trying to make where I was complaining about the tests not actually testing a code path?
      • raydeo
        yeah
      • x58
        That's cause so much was mocked out that it was incidentally testing it.
      • But in a way that didn't properly test that the functionality was correct
      • it just happened to hit the code path and thus be covered.
      • raydeo
        yeah not shipping a single integration test is a big no no to me
      • I agree using unit tests for all the corner cases is great... but you've gotta make sure the damn thing actually works
      • x58
        So would pyramid_tm even work on 1.3?
      • raydeo
        not atm!
      • well
      • x58
        I'm surprised we haven't had any complaints about it.
      • raydeo
        it's supposed to work on 1.3, dunno what's going on
      • but it's definitely broken on 1.2
      • despite us having tox explicitly test against 1.2 ;-)
      • x58
        special.
      • raydeo
        but it also seems like the request property isn't working on 1.3 :(
      • honestly not sure why 1.3 is broken atm
      • skalfyfan has quit
      • set_request_property seems to not be working or something
      • x58
        How are your getting your request object
      • ?
      • raydeo
        webtest
      • router
      • x58
        Gotcha.
      • raydeo
        test is literally just sending a request into a dummy app