search

theochem / cgrid

C++ version of horton (2.x) grid functionality
GNU General Public License v3.0
4 stars 1 forks source link

Travis release conditions #25

Closed matt-chan closed 6 years ago

matt-chan commented 6 years ago

Looks broken. Need to fix later.

tovrstra commented 6 years ago

@matt-chan Could you be more specific? E.g. a link to where it breaks and some description of the failure? Thanks.

matt-chan commented 6 years ago

Ah sorry! It was just a reminder for myself. I came across it a few minutes before giving a talk so I didn't have time to fix it.

I'll get rid of the bug today.

Matt

On Thu, Sep 21, 2017, 2:39 AM Toon Verstraelen notifications@github.com wrote:

@matt-chan https://github.com/matt-chan Could you be more specific? E.g. a link to where it breaks and some description of the failure? Thanks.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/theochem/qcgrids/issues/25#issuecomment-331065929, or mute the thread https://github.com/notifications/unsubscribe-auth/AA_-NWP5Zgxq2c4jEjUXr-LJTL4Yjauqks5skgSLgaJpZM4PeWJD .

-- Matt

Sent from my phone

matt-chan commented 6 years ago

Okay, so I should check before changing it @tovrstra https://github.com/theochem/qcgrids/blob/master/.travis.yml ln 223 shouldn't be there right?

Also we're missing a build for docs. Not sure if we have a project to be an example though.

tovrstra commented 6 years ago

Agreed. The condition on line 223 should not be there. As soon as qcgrids has some documentation, we can add a deployment for it.

matt-chan commented 6 years ago

Fixed in #28