Skip to content

[github] Implement prepare step#15382

Merged
seanshpark merged 1 commit intoSamsung:masterfrom
shs-park:github/cirintp-prepare-debian-release
May 15, 2025
Merged

[github] Implement prepare step#15382
seanshpark merged 1 commit intoSamsung:masterfrom
shs-park:github/cirintp-prepare-debian-release

Conversation

@shs-park
Copy link
Copy Markdown
Contributor

This adds the prepare step to run before building the circle-interpreter binary from the ONE compiler.

ONE-DCO-1.0-Signed-off-by: Seungho Henry Park shs.park@samsung.com

This adds the `prepare` step to run before building
the `circle-interpreter` binary from the ONE compiler.

ONE-DCO-1.0-Signed-off-by: Seungho Henry Park <shs.park@samsung.com>
@shs-park shs-park requested a review from a team May 15, 2025 11:36
@shs-park shs-park self-assigned this May 15, 2025
@shs-park shs-park added the PR/ready for review It is ready to review. Please review it. label May 15, 2025
@shs-park
Copy link
Copy Markdown
Contributor Author

This should be merged after #15380.

for issue #15213
from draft #15234

id: prepare
run: |
echo "TODO: Prepare and set distro versions"
VERSION="${{ needs.configure.outputs.version }}~${{ matrix.ubuntu_code }}"
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

${{ matrix.ubuntu_code }} needs #15380 to be landed first.

Copy link
Copy Markdown
Contributor

@seanshpark seanshpark left a comment

Choose a reason for hiding this comment

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

LGTM

@seanshpark seanshpark merged commit c161a75 into Samsung:master May 15, 2025
3 checks passed
@shs-park shs-park deleted the github/cirintp-prepare-debian-release branch May 19, 2025 05:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR/ready for review It is ready to review. Please review it.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants