Skip to content

Implement github workflow and fix profile commands if psql unavailable #976

Merged
jmchilton merged 7 commits intogalaxyproject:masterfrom
mvdbeek:gh_workflow
Dec 8, 2019
Merged

Implement github workflow and fix profile commands if psql unavailable #976
jmchilton merged 7 commits intogalaxyproject:masterfrom
mvdbeek:gh_workflow

Conversation

@mvdbeek
Copy link
Member

@mvdbeek mvdbeek commented Dec 8, 2019

No description provided.

@mvdbeek
Copy link
Member Author

mvdbeek commented Dec 8, 2019

@mvdbeek mvdbeek requested a review from jmchilton December 8, 2019 13:20
@jmchilton jmchilton merged commit 79ced29 into galaxyproject:master Dec 8, 2019
@jmchilton
Copy link
Member

Amazing!

@jmchilton
Copy link
Member

jmchilton commented Dec 8, 2019

Have you seen tox-gh-actions? galaxyproject/gxformat2@fea27f0

There isn't a ton of cruft doing it explicitly the way you have here - so I'm not sure it is worth introducing the black box magic tox-gh-actions does. But the magic is kind of cool at the same time.

Update: It is a cool way to skip running the doc tests and readme test on all but one profile, lint only on the earliest and latest version, and then actually run the tests on more versions? Something to consider

@martenson
Copy link
Member

It is a cool way to skip running the doc tests and readme test on all but one profile

The speed should be worth the blackbox I think. 👍

@mvdbeek
Copy link
Member Author

mvdbeek commented Dec 8, 2019

I don't think it is, we're talking seconds for readme compared to ~1 hour in total build time.

@martenson
Copy link
Member

Sorry, I am thinking Galaxy, where linting is long. 🙈

@mvdbeek
Copy link
Member Author

mvdbeek commented Dec 8, 2019

Still makes sense to look into it if we also want to test a wider range of python versions, we could skip the full-blown serve tests there

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.

4 participants