Skip to content

init: update the default sample Score file generated#53

Merged
mathieu-benoit merged 1 commit into
score-spec:mainfrom
Abhishek9639:feat/update-default-sample-score-file
Apr 27, 2026
Merged

init: update the default sample Score file generated#53
mathieu-benoit merged 1 commit into
score-spec:mainfrom
Abhishek9639:feat/update-default-sample-score-file

Conversation

@Abhishek9639
Copy link
Copy Markdown
Contributor

Description

The default score.yaml generated by score-helm init was using a basic stefanprodan/podinfo example. This PR replaces it with a more meaningful sample for consistency across all Score implementations.

What does this PR do?

Follow-up to score-spec/score-compose#476, score-spec/score-k8s#302, and score-spec/score-implementation-sample#103 as requested by @mathieu-benoit.

Updates the sample to use scorespec/sample-score-app:latest with postgres, dns, and route resources. Switched from struct-based YAML generation to a DefaultScoreFileContent raw string constant, which is simpler and supports comments.

Updated TestInitNominal and TestInitAndGenerate_with_sample accordingly.

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My change requires a change to the documentation.
  • I've signed off with an email address that matches the commit author.

Signed-off-by: Abhishek <abhishekup082@gmail.com>
@Abhishek9639
Copy link
Copy Markdown
Contributor Author

Hi @mathieu-benoit,
This is the last one score-helm is now updated too. All four repos (score-compose, score-k8s, score-implementation-sample, and score-helm) should now be consistent.
Please take a look when you get a chance.
Thanks

@mathieu-benoit mathieu-benoit self-requested a review April 27, 2026 19:27
Copy link
Copy Markdown
Contributor

@mathieu-benoit mathieu-benoit left a comment

Choose a reason for hiding this comment

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

LGTM

@mathieu-benoit mathieu-benoit merged commit 4e16e54 into score-spec:main Apr 27, 2026
5 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