Skip to content
This repository was archived by the owner on Aug 16, 2022. It is now read-only.

Add build testing to GH#27

Merged
romaninsh merged 11 commits intoatk4:masterfrom
mvorisek:test_in_gh_actions
Jan 19, 2021
Merged

Add build testing to GH#27
romaninsh merged 11 commits intoatk4:masterfrom
mvorisek:test_in_gh_actions

Conversation

@mvorisek
Copy link
Copy Markdown
Member

@mvorisek mvorisek commented Jan 15, 2021

fixes #18

@mvorisek mvorisek marked this pull request as ready for review January 15, 2021 19:21
@mvorisek mvorisek mentioned this pull request Jan 16, 2021
Copy link
Copy Markdown
Member

@romaninsh romaninsh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved - but please add more info on running "make.php" into README - for local testing.


## Running Locally

Run `make build` to regenerate Dockerfiles and to rebuild all docker images. Check "Makefile" for more details
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if you are replacing make with "make.php" could you mention that in the README instead?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@romaninsh romaninsh merged commit 68f0378 into atk4:master Jan 19, 2021
@mvorisek mvorisek deleted the test_in_gh_actions branch January 19, 2021 14:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Fix pipeline to still build on PRs (even by third parties), but don't tag images.

2 participants