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

[bug] score-compose init --provisioners flag doesn't document local file option and doesn't support stdin #220

Open
astromechza opened this issue Dec 6, 2024 · 3 comments · May be fixed by #221
Assignees
Labels
bug Something isn't working documentation Improvements or additions to documentation go Pull requests that update Go code good first issue Good for newcomers help wanted Extra attention is needed

Comments

@astromechza
Copy link
Member

the docs currently show

      --provisioners stringArray   Provisioner files to install. May be specified multiple times. Supports:
                                   - HTTP        : http://host/file
                                   - HTTPS       : https://host/file
                                   - Git (SSH)   : git-ssh://git@host/repo.git/file
                                   - Git (HTTPS) : git-https://host/repo.git/file
                                   - OCI         : oci://[registry/][namespace/]repository[:tag|@digest][#file]

But it also supports local file paths as well!

Also it would be very conventient to also support - as stdin so that we can run:

score-compose init --provisioners=- <<EOF
- uri: template://custom-env
  type: environment
  outputs:
    FIZZ: buzz
EOF
@astromechza astromechza added bug Something isn't working documentation Improvements or additions to documentation good first issue Good for newcomers go Pull requests that update Go code labels Dec 6, 2024
@mathieu-benoit mathieu-benoit added the help wanted Extra attention is needed label Dec 6, 2024
@rabelmervin
Copy link

Hi @astromechza Can i contribute to this issue ? This will be a good learning opportunity for me : )

@mathieu-benoit
Copy link
Contributor

Sure thing, @rabelmervin, please go ahead, thanks!

@rabelmervin
Copy link

rabelmervin commented Dec 13, 2024

Hi @mathieu-benoit @astromechza I have made a pr. Could you please review my changes ?

@mathieu-benoit mathieu-benoit linked a pull request Dec 13, 2024 that will close this issue
@7h3-3mp7y-m4n 7h3-3mp7y-m4n linked a pull request Dec 16, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation go Pull requests that update Go code good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants