Skip to content

Commit

Permalink
Merge pull request #150 from concordia-fsae/user/jl/use-reusable-work…
Browse files Browse the repository at this point in the history
…flows

User/jl/use reusable workflows
  • Loading branch information
JoshLafleur authored Mar 9, 2025
2 parents e8e2d9e + 5db4458 commit fee1b27
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .github/pull_request_template.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,4 +8,4 @@

### Test Plan

1. Describe tests needed to validate the software changes
- [ ] Describe tests needed to validate the software changes
8 changes: 4 additions & 4 deletions .github/workflows/reusable/build.yml
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
name: Build Platform and Archive
name: Build and Archive

on:
workflow_dispatch:
inputs:
platform:
build-targets:
required: true
type: string
flags:
Expand Down Expand Up @@ -38,10 +38,10 @@ jobs:
- name: Get Clean Branch
id: pull
uses: concordia-fsae/firmware/.github/workflows/reusable/clean_pull.yml@master
- name: Build Platform ${{ inputs.platform }}
- name: Build Targets
id: build
run: |
scons --${{ inputs.build-type }}=${{ inputs.platform }} ${{ inputs.flags }}
scons --${{ inputs.build-type }}=${{ inputs.build-targets }} ${{ inputs.flags }}
- name: Archive Artifacts
id: archive
uses: actions/upload-artifact@v4
Expand Down

0 comments on commit fee1b27

Please sign in to comment.