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

Update Chromium Example #896

Merged
merged 1 commit into from
May 1, 2024

Conversation

nfarah86
Copy link
Contributor

@nfarah86 nfarah86 commented Apr 30, 2024

Description

Updated Chromium example to include the Docker engine in the Readme, updated script 3, and included the slack community if devs need help

Fixes # (issue)

Type of change

Please delete options that aren't relevant.

  • This change requires a documentation update

Checklist

  • Updated documentation if needed
  • Tests added/amended
  • [x ] PR is contained in a single commit, using git amend see some docs

This change is Reviewable

Copy link
Contributor

@adam-singer adam-singer left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: :shipit: complete! 1 of 1 LGTMs obtained

Copy link
Collaborator

@MarcusSorealheis MarcusSorealheis left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewable status: :shipit: complete! 2 of 1 LGTMs obtained

@nfarah86 nfarah86 merged commit 185eab3 into TraceMachina:main May 1, 2024
24 checks passed
chinchaun pushed a commit to chinchaun/nativelink that referenced this pull request May 6, 2024
Co-authored-by: nadine hachouche <nadinehachouche@nadines-MacBook-Pro.local>
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.

3 participants