Skip to content

Add github actions testing - #109

Closed
skjnldsv wants to merge 5 commits into
masterfrom
add/actions
Closed

Add github actions testing#109
skjnldsv wants to merge 5 commits into
masterfrom
add/actions

Conversation

@skjnldsv

@skjnldsvskjnldsv commented Feb 3, 2020

Copy link
Copy Markdown
Member

Add testing

@rullzer I feel like adding those here doesn't really defined themselves as the 'standard actions' files :)

Maybe we should have a repo for it?

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
Comment thread.github/workflows/node.yml
@skjnldsv
skjnldsvforce-pushed the add/actions branch 2 times, most recently from 1174013 to 303803aCompareFebruary 3, 2020 13:09
@skjnldsv
skjnldsvforce-pushed the add/actions branch 21 times, most recently from 1d3d46e to c45ab96CompareFebruary 3, 2020 16:13
@skjnldsv
skjnldsvforce-pushed the add/actions branch 2 times, most recently from 0576665 to 5f38cf1CompareFebruary 3, 2020 17:20
Comment thread.github/workflows/phpunit.yml Outdated
@skjnldsv
skjnldsvforce-pushed the add/actions branch 23 times, most recently from 0ad2a94 to 4eff004CompareFebruary 4, 2020 09:51
@skjnldsv

skjnldsv commented Feb 4, 2020

Copy link
Copy Markdown
MemberAuthor

I give up, somehow the connexion is ok, because the nextcloud setup is working properly for the first part.
But Nextcloud always create its own admin db user, and once it creates it (oc_admin, it works) it can't connect with it afterwards.

@skjnldsv

Copy link
Copy Markdown
MemberAuthor

So, now that server is fixed nextcloud/server#19303
I can move on :)

Ideally since github actions doesn't support services.if, but only steps.if, I cannot start services based on what db matrix is being used. So we'd need to split things in the meantime :/

There are a few steps actions like https://github.com/Harmon758/postgresql-action & https://github.com/mirromutth/mysql-action but it seems to have a few bugs (mysql) or missing features (port setting pgsql)

@skjnldsv

Copy link
Copy Markdown
MemberAuthor

/compile amend /

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
Signed-off-by: npmbuildbot[bot] <npmbuildbot[bot]@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@skjnldsv@ChristophWurst@juliusknorr