Skip to content

✨ Create Zest Build Action #1

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
Merged
SizzinSeal merged 63 commits into from
Mar 18, 2025
Merged

✨ Create Zest Build Action #1

SizzinSeal merged 63 commits into from
Mar 18, 2025

Conversation

@ABUCKY0
Copy link
Contributor

@ABUCKY0 ABUCKY0 commented Mar 17, 2025 •
edited
Loading

Create Zest Build action based on LemLib/pros-build.

Final To-Do before merging:

  • Change image source in action.yml from Dockerfile to a container on ghcr.io
  • Pass Reviews (duh)

ABUCKY0 added 30 commits March 12, 2025 17:01
@ABUCKY0
feat: ✨ Initial Commit - Add initial build action and Docker setup fo…
14f4af1 
…r ZestCode templates
@ABUCKY0
feat: ✨ Update workflows to do moer autonomously and reset build.sh b…
4c71528 
…ack to the two basic commands.
@ABUCKY0
fix: ✨ Update repository, ref, and caller_token inputs to work.
17f76c4
@ABUCKY0
feat: ✨ Update test workflow to use Docker for action execution
963d667
@ABUCKY0
fix: 🐛 Maybe fix Error: Repository path '/ZestCodeCloned' is not unde…
3ed09ed 
…r '/home/runner/work/build-action/build-action'
@ABUCKY0
fix: 🐛 Correct directory in test workflow
65b92fe
@ABUCKY0
test: list all files in the directory
addb466
@ABUCKY0
test: 🐛 Remove meson build commands so list directory command may act…
f89ff19 
…ually work
@ABUCKY0
fix: 🐛 Update Docker command in test workflow to actually change entr…
849fb0b 
…ypoint
@ABUCKY0
fix: 🐛 Change docker run command to maybe correct path
bfbe1a9
@ABUCKY0
fix: 🐛 Update paths in test workflow for Docker command
1c0f1ba
@ABUCKY0
fix: 🐛 Update Docker paths in test workflow
deec299
@ABUCKY0
refactor: ✨ Replace shell script with Python build script and update …
9c20f51 
…Dockerfile
@ABUCKY0
fix: 🐛 Change Dockerfile entrypoint to CMD instead of RUN
cfeb824
@ABUCKY0
ci: 🐛 Fix continue-on-error input not existing by manually setting co…
2da63d3 
…ntinue-on-error in test.yml
@ABUCKY0
fix: 🐛 Initialize setup_output and compile_output before try-catch st…
24af438 
…atements in build script
@ABUCKY0
fix: 🐛 Update LICENSE path in Dockerfile and build script; declare va…
1e1ef10 
…riables as global in build.py
@ABUCKY0
fix: 🐛 Correctly declare global variables
ae1ed09
@ABUCKY0
fix: 🐛 Actually fix global variables
261ac2c
@ABUCKY0
fix: 🐛 Declare global variables for setup_output, compile_output, bui…
7ea6d5b 
…ld_finish_time, and build_duration
@ABUCKY0
fix: 🐛 Test to see if the python script is running meson or otherwise
10671b9
@ABUCKY0
fix: 🐛 Capture output in meson setup command instead of redirecting t…
88b6574 
…o stdout and stderr
@ABUCKY0
fix: 🐛 Enable unbuffered output for Python and capture output in meso…
af85db0 
…n compile command
@ABUCKY0
fix: 🐛 Print stdout and stderr output for meson setup and compile com…
4215bc4 
…mands
@ABUCKY0
fix: 🐛 Ensure stdout and stderr are printed after meson setup and com…
bf64d92 
…pile commands
@ABUCKY0
fix: 🐛 Update meson setup and compile commands to handle errors and p…
0a950d2 
…rint output correctly
@ABUCKY0
fix: 🐛 Remove unnecessary decoding of stdout output in build script
abc17f7
@ABUCKY0
refactor: 🔥 Switch back from python to bash shell script
adedba9
@ABUCKY0
ci: 🐛 switch to main from build/meson-init in test action
7b2bad6
@ABUCKY0
fix: 🐛 Correctly capture exit code from meson setup for error handling
656754f
ABUCKY0 added 19 commits March 13, 2025 23:59
@ABUCKY0
fix: 🐛 Fix Step Summary not being written by removing definiton of $G…
8ac9f12 
…ITHUB_STEP_SUMMARY in the script
@ABUCKY0
fix: 🐛 Use cat instead of echo to write stdout output to summary.
5c29886
@ABUCKY0
fix: 🐛 Add spacing to GitHub step summary so formatting will actually…
90bd2b7 
… occur
@ABUCKY0
feat: ✨ add write_job_summary input
4fadc66
@ABUCKY0
fix: 🐛 Replace alias with functions for error handling in build script
89a51d8
@ABUCKY0
ci: 🐛 (Maybe?) Correct syntax error in build-image.yml
12343ee
@ABUCKY0
revert: 💚 Undo syntax change since it caused a different issue
1dc969b
@ABUCKY0
ci: 💚 Maybe fix build-image workflow (fr this time)
43a2c76
@ABUCKY0
ci: 💚 Correct repository format in build-image.yml, possibly fixing b…
8c8c4f0 
…uild-image workflow
@ABUCKY0
ci: 💚 fix CI build (Hopefully actually fr this time)
72186d6
@ABUCKY0
fix: 🐛 Make the write_job_summary input actually work
a73f1b5
@ABUCKY0
fix: 🐛 Fix conditional check for INPUT_WRITE_JOB_SUMMARY
18516fb
@ABUCKY0
fix: 🐛 Add write_job_summary input to Test Action in test.yml
18e610e
@ABUCKY0
test: 🐛 Add debugging output for Meson setup exit code in build.sh
3920da6
@ABUCKY0
docs: 📝 Start filling out README.md
e731f34
@ABUCKY0
ci: ✨ Add Upload_Image job to build-image.yml and upload binary in te…
5981594 
…st.yml
@ABUCKY0
ci: 🔥 Disable binary uploading step
a10e40d
@ABUCKY0
ci: ✨ Update job names in build-image.yml and test.yml
db0fb07
@ABUCKY0
ci: ✨ Update job name format in test.yml for clarity
4ba928d
@ABUCKY0 ABUCKY0 marked this pull request as ready for review March 17, 2025 01:12
@ABUCKY0 ABUCKY0 self-assigned this Mar 17, 2025
@ABUCKY0 ABUCKY0 requested a review from meisZWFLZ March 17, 2025 01:12
@ABUCKY0 ABUCKY0 mentioned this pull request Mar 17, 2025
Open
ion098
ion098 reviewed Mar 17, 2025
View reviewed changes
@ABUCKY0 ABUCKY0 requested a review from SizzinSeal March 18, 2025 03:48
SizzinSeal
SizzinSeal approved these changes Mar 18, 2025
View reviewed changes
Copy link
Member

@SizzinSeal SizzinSeal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@SizzinSeal SizzinSeal merged commit a38c5db into main Mar 18, 2025
8 checks passed
@SizzinSeal SizzinSeal deleted the move/pros-build-to-zest-build branch March 18, 2025 20:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ion098 ion098 ion098 left review comments

@SizzinSeal SizzinSeal SizzinSeal approved these changes

@meisZWFLZ meisZWFLZ Awaiting requested review from meisZWFLZ

Assignees

@ABUCKY0 ABUCKY0

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ABUCKY0 @SizzinSeal @ion098