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.

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

refactor(workflows): migrate report after build #199

Merged
merged 3 commits into from
Dec 11, 2024

Conversation

0x1026
Copy link
Member

@0x1026 0x1026 commented Dec 11, 2024

This pull request includes significant changes to the GitHub Actions workflows, particularly focusing on the build, continuous integration (CI), and continuous deployment (CD) processes. The main changes involve updating input parameters, modifying job strategies, and restructuring the workflows to improve efficiency and clarity.

Workflow Updates:

  • Build Workflow:

    • Updated input parameters to include context, environment, and image instead of images. (.github/workflows/build.yml)
    • Removed the matrix strategy and directly used input parameters for the environment and image. (.github/workflows/build.yml)
    • Adjusted Docker metadata-action and build-test steps to use inputs.image and inputs.context. (.github/workflows/build.yml) [1] [2] [3] [4]
  • Continuous Deployment (CD) Workflow:

    • Reintroduced the matrix strategy to handle multiple images and contexts. (.github/workflows/cd.yml)
  • Continuous Integration (CI) Workflow:

    • Adjusted the outputs and reintroduced the matrix strategy for handling multiple images and contexts. (.github/workflows/ci.yml) [1] [2] [3]
  • Report Workflow:

    • Removed the report.yml workflow and integrated its functionality directly into the build workflow. (.github/workflows/report.yml)

@github-actions github-actions bot added the kind/improvement Categorizes issue or PR as related to an improvement. label Dec 11, 2024
@0x1026 0x1026 marked this pull request as ready for review December 11, 2024 00:25
@0x1026 0x1026 requested a review from a team as a code owner December 11, 2024 00:25
@0x1026 0x1026 requested a review from albert1413 December 11, 2024 00:25
@0x1026 0x1026 self-assigned this Dec 11, 2024
@0x1026 0x1026 added this to the Sprint 3 - Prototype milestone Dec 11, 2024
@0x1026 0x1026 merged commit 20eb394 into main Dec 11, 2024
14 checks passed
@0x1026 0x1026 deleted the refactor/migrate-report-after-build branch December 11, 2024 00:26
0x1026 added a commit that referenced this pull request Dec 11, 2024
This pull request includes significant changes to the GitHub Actions
workflows, particularly focusing on the build, continuous integration
(CI), and continuous deployment (CD) processes. The main changes involve
updating input parameters, modifying job strategies, and restructuring
the workflows to improve efficiency and clarity.

### Workflow Updates:

* **Build Workflow:**
* Updated input parameters to include `context`, `environment`, and
`image` instead of `images`. (`.github/workflows/build.yml`)
* Removed the matrix strategy and directly used input parameters for the
environment and image. (`.github/workflows/build.yml`)
* Adjusted Docker metadata-action and build-test steps to use
`inputs.image` and `inputs.context`. (`.github/workflows/build.yml`)
[[1]](diffhunk://#diff-5c3fa597431eda03ac3339ae6bf7f05e1a50d6fc7333679ec38e21b337cb6721L66-R61)
[[2]](diffhunk://#diff-5c3fa597431eda03ac3339ae6bf7f05e1a50d6fc7333679ec38e21b337cb6721L101-R112)
[[3]](diffhunk://#diff-5c3fa597431eda03ac3339ae6bf7f05e1a50d6fc7333679ec38e21b337cb6721L127-R122)
[[4]](diffhunk://#diff-5c3fa597431eda03ac3339ae6bf7f05e1a50d6fc7333679ec38e21b337cb6721L156-R192)

* **Continuous Deployment (CD) Workflow:**
* Reintroduced the matrix strategy to handle multiple images and
contexts. (`.github/workflows/cd.yml`)

* **Continuous Integration (CI) Workflow:**
* Adjusted the outputs and reintroduced the matrix strategy for handling
multiple images and contexts. (`.github/workflows/ci.yml`)
[[1]](diffhunk://#diff-b803fcb7f17ed9235f1e5cb1fcd2f5d3b2838429d4368ae4c57ce4436577f03fL24-R24)
[[2]](diffhunk://#diff-b803fcb7f17ed9235f1e5cb1fcd2f5d3b2838429d4368ae4c57ce4436577f03fL38-R51)
[[3]](diffhunk://#diff-b803fcb7f17ed9235f1e5cb1fcd2f5d3b2838429d4368ae4c57ce4436577f03fL48-R61)

* **Report Workflow:**
* Removed the `report.yml` workflow and integrated its functionality
directly into the build workflow. (`.github/workflows/report.yml`)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/improvement Categorizes issue or PR as related to an improvement.
Projects
Status: Fet
Development

Successfully merging this pull request may close these issues.

1 participant