Skip to content

Conversation

@ruimartinsptl
Copy link
Contributor

No description provided.

Copy link
Member

@Nfsaavedra Nfsaavedra left a comment

Choose a reason for hiding this comment

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

Nice! Thanks this is useful. Just two things:

  1. Can you add a mention to this in the README?
  2. Ruby needs to be installed so that the parser for Chef works

@ruimartinsptl
Copy link
Contributor Author

Just updated, feel free to suggest changes. I splitted the instruction in README in two parts INSTALL and USAGE, do you prefer in a different way?

@ruimartinsptl
Copy link
Contributor Author

Ah, one more thing, I pushed the image for my dockerhub account: https://hub.docker.com/repository/docker/ruimartinsptl/sr-lab-glitch/general

If you create an sr-lab account in dockerhub, we can create an official imagem there :) (because for now people need to execute the docker build locally, or use my image ruimartinsptl/sr-lab-glitch, but is not official)

@Nfsaavedra
Copy link
Member

Ah, one more thing, I pushed the image for my dockerhub account: https://hub.docker.com/repository/docker/ruimartinsptl/sr-lab-glitch/general

If you create an sr-lab account in dockerhub, we can create an official imagem there :) (because for now people need to execute the docker build locally, or use my image ruimartinsptl/sr-lab-glitch, but is not official)

Yes that is a good idea! We'll work on that soon, I'm not sure if there is a sr-lab account yet. However I think it is also not a big deal right now to build the image locally as the image is really small and consequently really fast to build.

Copy link
Member

@Nfsaavedra Nfsaavedra left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you, I'll merge the PR

@Nfsaavedra Nfsaavedra merged commit 19c5020 into sr-lab:main Oct 27, 2025
2 checks passed
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