Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Build the Action as a Container #6

Merged
merged 8 commits into from
Apr 13, 2022
Merged

Build the Action as a Container #6

merged 8 commits into from
Apr 13, 2022

Conversation

damacus
Copy link
Contributor

@damacus damacus commented Feb 24, 2022

Description

  • Build the Action as a Container to save people from building it every time an Action is run.
  • Remove the entrypoint and run it all from the Docker container
  • Format the output as GitHub output

Issues Resolved

None

Check List

  • New functionality includes testing.
  • New functionality has been documented in the README if applicable.

@damacus
Build the Action as a Container to save people from building it every
5e9ed95 
time

Signed-off-by: Dan Webb <dan.webb@damacus.io>
@damacus damacus requested a review from xorima February 24, 2022 11:12
@damacus
Add build action
9704116 
Signed-off-by: Dan Webb <dan.webb@damacus.io>
@damacus
Fix folders
856d511 
Signed-off-by: Dan Webb <dan.webb@damacus.io>
@damacus
Fix stage name
fc90b32
@damacus
Try a full directory path for the matchers
dcb36b8 
Signed-off-by: Dan Webb <dan.webb@damacus.io>
@damacus
Remove src from directory
bf15879 
Signed-off-by: Dan Webb <dan.webb@damacus.io>
@damacus
Update add-mater to use ::
a01f797 
As per actions/toolkit#59

Signed-off-by: Dan Webb <dan.webb@damacus.io>
@damacus
Bump Action version
1d5953c 
Signed-off-by: Dan Webb <dan.webb@damacus.io>
@damacus damacus merged commit a1c374b into main Apr 13, 2022
@damacus damacus deleted the build-container branch April 13, 2022 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xorima xorima xorima approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@damacus @xorima