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

Fix the Manifest Parser #13

Merged
merged 40 commits into from
Mar 4, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
40 commits
Select commit Hold shift + click to select a range
3f82a27
Remove Non-Submodule
Byrdman32 Mar 3, 2024
6674a64
Add Submodule
Byrdman32 Mar 4, 2024
feb6c69
Remove Old Docker File
Byrdman32 Mar 4, 2024
ed838a3
Switch Branch
Byrdman32 Mar 4, 2024
5c95ff8
Update Script
Byrdman32 Mar 4, 2024
7efff8a
Fix Function Renames
Byrdman32 Mar 4, 2024
af44079
Update Manifest
Byrdman32 Mar 4, 2024
db1a1a1
Don't allow user edits of the manifest
Byrdman32 Mar 4, 2024
636c700
Fix Manifest Action
Byrdman32 Mar 4, 2024
cafa969
Fix Manifest Action
Byrdman32 Mar 4, 2024
4dcf453
Fix Manifest Action
Byrdman32 Mar 4, 2024
c132f32
Change Trigger
Byrdman32 Mar 4, 2024
d11ec55
Add another trigger
Byrdman32 Mar 4, 2024
75b890f
Fix Paths
Byrdman32 Mar 4, 2024
195af4a
Fix Fetch
Byrdman32 Mar 4, 2024
d9c481b
Fix Commit Path
Byrdman32 Mar 4, 2024
24f051d
[TEMP] Change to master branch of manifest
Byrdman32 Mar 4, 2024
9c3f26e
Fix Manifest Parser and Continued Temp
Byrdman32 Mar 4, 2024
766ffb8
Find Branch Name
Byrdman32 Mar 4, 2024
68bc6ba
Fix Push Destination
Byrdman32 Mar 4, 2024
dac8a2a
Another Fix
Byrdman32 Mar 4, 2024
cf55069
More Fixes
Byrdman32 Mar 4, 2024
661d739
Reverse Steps
Byrdman32 Mar 4, 2024
75730d0
Different Fix?
Byrdman32 Mar 4, 2024
0bf58c7
Try old method
Byrdman32 Mar 4, 2024
619b8ae
Fix Push Destination
Byrdman32 Mar 4, 2024
a41a6c2
Find Target Name
Byrdman32 Mar 4, 2024
5eaa364
Different Base
Byrdman32 Mar 4, 2024
6c6812b
Fix?
Byrdman32 Mar 4, 2024
4b5cb65
Fix?
Byrdman32 Mar 4, 2024
61e52be
Reorder
Byrdman32 Mar 4, 2024
53c1116
** Automated - Manifest Update **
MRDT-Admin Mar 4, 2024
4b33566
Undo Temp Changes
Byrdman32 Mar 4, 2024
5093b8b
Merge branch 'hotfix/rovecomm-parser' of https://github.com/MissouriM…
Byrdman32 Mar 4, 2024
d7f693e
** Automated - Manifest Update **
MRDT-Admin Mar 4, 2024
ad9f994
Re-undo Temp Changes
Byrdman32 Mar 4, 2024
2f9bb82
Merge branch 'hotfix/rovecomm-parser' of https://github.com/MissouriM…
Byrdman32 Mar 4, 2024
171e56d
** Automated - Manifest Update **
MRDT-Admin Mar 4, 2024
8cd6e75
Re-Re-Undo Temp Changes
Byrdman32 Mar 4, 2024
86b2a89
** Automated - Manifest Update **
MRDT-Admin Mar 4, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
104 changes: 0 additions & 104 deletions .devcontainer/Dockerfile_Focal

This file was deleted.

3 changes: 1 addition & 2 deletions .devcontainer/devcontainer.json
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,7 @@
"-e DISPLAY" // Makes DISPLAY environment variable available in the container.
// "-v /tmp/.X11-unix:/tmp/.X11-unix" // Required for container to access display.
],
// "image": "ghcr.io/missourimrdt/rovecomm-jammy:latest",
"image": "ghcr.io/missourimrdt/rovecomm-focal:latest",
"image": "ghcr.io/missourimrdt/rovecomm-jammy:latest",
// "image": "ghcr.io/missourimrdt/rovecomm-jetpack:latest",
// "build": {
// "dockerfile": "Dockerfile_Jammy"
Expand Down
49 changes: 30 additions & 19 deletions .github/workflows/manifest.yml
Original file line number Diff line number Diff line change
@@ -1,37 +1,48 @@
name: Update Manifest Header

on:
issue_comment:
types: [created, edited, deleted]
pull_request:
branches: ["development"]
paths:
- "data/RoveComm/manifest.json"
- "tools/RoveComm/parser.py"
- "src/RoveComm/RoveCommManifest.h"
- ".github/workflows/manifest.yml"

concurrency:
group: "manifest-generator"
cancel-in-progress: true

jobs:
update-manifest:
if: ${{ github.event.issue.pull_request && contains(github.event.comment.body, '/update-manifest') }}
runs-on: ubuntu-latest
steps:
- name: Extract branch name
shell: bash
run: echo "branch=${GITHUB_HEAD_REF:-${GITHUB_REF#refs/heads/}}" >> $GITHUB_OUTPUT
id: extract_branch

- name: Update Branch
uses: actions/checkout@v3
- name: Pull Repo
uses: actions/checkout@v4
with:
fetch-depth: 0
token: ${{ github.token }}
ref: ${{ steps.extract_branch.outputs.branch }}
submodules: "recursive"

- name: Setup Git
run: |
git config user.name "MissouriMRDT"
git config user.email "[email protected]"

- name: Rebase Changes
run: |
git fetch origin ${{ github.head_ref }}
git rebase origin/${{ github.head_ref }}

- name: Generate Manifest and Commit
- name: Generate Manifest
run: |
git branch
cd tools/RoveComm
python parser.py
git config user.name "MissouriMRDT"
git config user.email "[email protected]"
git add -f ../../src/RoveComm/Manifest.h
git commit -m "** Automated - Manifest Update **"
git push

- name: Commit Manifest
run: |
git add -f src/RoveComm/RoveCommManifest.h
git commit -m "** Automated - Manifest Update **" || echo "No changes to commit"

- name: Push Changes
run: |
git push origin HEAD:${{ github.head_ref }} --force-with-lease || echo "Failed to push changes"
2 changes: 1 addition & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ valgrind.rpt
### RoveComm ###\
# Note: To limit local edits that get pushed and then overridden, we include this in the .gitignore.
# However, the GitHub Action will be allowed to push by bypassing the .gitignore.
Manifest.h
RoveCommManifest.h

# Documentation. This is a special rule that uses ! negations.
docs/*
Expand Down
4 changes: 4 additions & 0 deletions .gitmodules
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
[submodule "data/RoveComm"]
path = data/RoveComm
url = https://github.com/MissouriMRDT/RoveComm_Base
branch = embedded-2024
1 change: 1 addition & 0 deletions data/RoveComm
Submodule RoveComm added at 78425a
Loading