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

Changes made for provisoners flag #221

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rabelmervin
Copy link

Initially the provisioners flag doesnt support local file and stdin. But now local and stdin added

@mathieu-benoit
Copy link
Contributor

Thanks, @rabelmervin!

You need to sign your commits to have the DCO passing successfully on this PR:

To add your Signed-off-by line to every commit in this branch:
Ensure you have a local copy of your branch by checking out the pull request locally via command line.
In your local branch, run: git rebase HEAD~1 --signoff
Force push your changes to overwrite the branch: git push --force-with-lease origin new

Signed-off-by: rabelmervin <[email protected]>
@rabelmervin
Copy link
Author

Hello @mathieu-benoit @astromechza I have signed the commits and DCO is passed successfully :) Is there anything i can help you with ?

@7h3-3mp7y-m4n
Copy link
Contributor

7h3-3mp7y-m4n commented Dec 14, 2024

hey, @rabelmervin I'm not a reviewer but can you also update the test case over here https://github.com/score-spec/score-compose/blob/main/internal/command/init_test.go#L79
Can you also add the test case for your function?

@@ -0,0 +1,35 @@
# Score provides a developer-centric and platform-agnostic
Copy link
Contributor

@7h3-3mp7y-m4n 7h3-3mp7y-m4n Dec 16, 2024

Choose a reason for hiding this comment

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

I think @rabelmervin you should not add this file cause it's irrelevant to issue #220
Also, it is of no use.

Comment on lines +1 to +35
# Score provides a developer-centric and platform-agnostic
# Workload specification to improve developer productivity and experience.
# Score eliminates configuration management between local and remote environments.
#
# Specification reference: https://docs.score.dev/docs/reference/score-spec-reference/
---

# Score specification version
apiVersion: score.dev/v1b1

metadata:
name: example

containers:
hello-world:
image: nginx:latest

# Uncomment the following for a custom entrypoint command
# command: []

# Uncomment the following for custom arguments
# args: []

# Environment variables to inject into the container
variables:
EXAMPLE_VARIABLE: "example-value"

service:
ports:
# Expose the http port from nginx on port 8080
www:
port: 8080
targetPort: 80

resources: {}
Copy link
Contributor

Choose a reason for hiding this comment

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

Same for this, If you want to add examples, you should refer to this and in a separate PR.

@rabelmervin
Copy link
Author

rabelmervin commented Dec 16, 2024

Hi @7h3-3mp7y-m4n I'll the add the test functions and I'll remove that irrelevant files 👍

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.

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