Skip to content

update workflow and makefile#5

Merged
zotoMIT merged 1 commit intomainfrom
makefile-actions-update
Jun 2, 2022
Merged

update workflow and makefile#5
zotoMIT merged 1 commit intomainfrom
makefile-actions-update

Conversation

@zotoMIT
Copy link
Collaborator

@zotoMIT zotoMIT commented Jun 2, 2022

Subject

Applies a universal github workflow and standard makefile commands to the app.

Why these changes are being introduced:

This commit is part of a larger refactor of how we handle ECR repositories and generating makefile and github workflows.
The work here will likely be removed when stage and prod are deployed, so this is a temporary change, but serves as a proof of concept/stopgap until stage is ready.
The changes here represent a direct copy/paste from the infrastructure repository.

How this addresses that need:

Add the dev-build workflow
Update the makefile with our new standard format

Side effects of this change:

A test of this workflow will happen when I submit the PR for this.

Relevant ticket(s):

NA

# Subject
Applies a universal github workflow and standard makefile commands to the app.

# Why these changes are being introduced:
This commit is part of a larger refactor of how we handle ECR repositories and generating makefile and github workflows.
The work here will likely be removed when stage and prod are deployed, so this is a temporary change, but serves as a proof of concept/stopgap until stage is ready.
The changes here represent a direct copy/paste from the infrastructure repository.

# How this addresses that need:
Add the dev-build workflow
Update the makefile with our new standard format

# Side effects of this change:
A test of this workflow will happen when I submit the PR for this.

# Relevant ticket(s):
NA
@zotoMIT zotoMIT requested a review from ehanson8 June 2, 2022 18:26
@zotoMIT zotoMIT self-assigned this Jun 2, 2022
@zotoMIT zotoMIT merged commit 2388d38 into main Jun 2, 2022
@zotoMIT zotoMIT deleted the makefile-actions-update branch June 2, 2022 19:30
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.

2 participants