8000 Caddy base runtime by V1ad8 · Pull Request #177 · unikraft/catalog · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Caddy base runtime #177

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

Merged
merged 2 commits into from
May 10, 2025
Merged

Caddy base runtime #177

merged 2 commits into from
May 10, 2025

Conversation

V1ad8
Copy link
Contributor
@V1ad8 V1ad8 commented Apr 26, 2025

Added a new version of the caddy example that uses the base runtime.

Copy link
Contributor
@razvand razvand left a comment

Choose a reason for hiding this comment

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

See my comments. Improve your commit message. Use present tense. Use Caddy instead of caddy. Add rationale / reason for using base (to increase transparency in the contents of the application filesystem and the build steps).

Read carefully and follow rigorously the instructions into doing good Git commits.

See also the commit history in the catalog repository and follow the same syntax and contents as those.

Be careful, attentive and rigorous.

@V1ad8 V1ad8 requested a review from razvand April 28, 2025 20:07
@razvand razvand self-assigned this May 10, 2025
@V1ad8 V1ad8 force-pushed the cdl-runtime branch 2 times, most recently from fa735d4 to 5e08ac2 Compare May 10, 2025 10:22
Copy link
Contributor
@razvand razvand left a comment

Choose a reason for hiding this comment

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

Reviewed-by: Razvan Deaconescu razvand@unikraft.io
Approved-by: Razvan Deaconescu razvand@unikraft.io

@V1ad8 V1ad8 force-pushed the cdl-runtime branch 2 times, most recently from f5c0847 to e59ac07 Compare May 10, 2025 10:33
V1ad8 added 2 commits May 10, 2025 13:43
Introduce a new version of the Caddy example that uses the base runtime
The base example is used to increase transparency in the contents of the
application filesystem and the build steps.

Signed-off-by: Vlad-Marin UNGUREANU <vlad.ungureanu1808@stud.acs.upb.ro>
Introduced a new version of the Node18 example that uses the base runtime.
The base version is used to increase transparency in the contents of the
application filesystem and the build steps.

Signed-off-by: Vlad-Marin UNGUREANU <vlad.ungureanu1808@stud.acs.upb.ro>
@razvand razvand merged commit 68d886c into unikraft:main May 10, 2025
@V1ad8 V1ad8 mentioned this pull request May 17, 2025
17 tasks
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
0