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

Moving proto-build to ACAP runtime #96

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

johan-olsson-work
Copy link
Contributor

@johan-olsson-work johan-olsson-work commented Sep 2, 2024

Describe your changes

This PR includes both #77: Adding a new dockerfile to ACAP runtime that builds proto APIs for python.
and
PR #86: Adding a workflow file to build and verify the image from PR above.

Issue ticket number and link

  • Fixes #(issue)

Checklist before requesting a review

  • I have performed a self-review of my own code
  • I have verified that the code builds perfectly fine on my local system
  • I have commented my code, particularly in hard-to-understand areas
  • I have verified that my code follows the style already available in the repository
  • I have made corresponding changes to the documentation

Dockerfile.proto Outdated Show resolved Hide resolved
Dockerfile.proto Outdated Show resolved Hide resolved
Dockerfile.proto Outdated Show resolved Hide resolved
Dockerfile.proto Outdated Show resolved Hide resolved
Dockerfile.proto Outdated Show resolved Hide resolved
Dockerfile.proto Outdated Show resolved Hide resolved
@johan-olsson-work johan-olsson-work force-pushed the squaching-angelos-proto-work branch from 374f894 to 444a334 Compare September 10, 2024 13:04
@johan-olsson-work johan-olsson-work force-pushed the squaching-angelos-proto-work branch from 444a334 to 51149c3 Compare September 10, 2024 13:09
@madelen-at-work madelen-at-work merged commit 511e616 into main Sep 12, 2024
13 checks passed
@madelen-at-work madelen-at-work deleted the squaching-angelos-proto-work branch September 12, 2024 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants