Skip to content

[Devfile#1721] Adding renovate.json and test script to ensure sample can still be parsed and used by consumers for go sample#45

Merged
JslYoon merged 4 commits into
devfile-samples:mainfrom
JslYoon:devfile#1721
Jul 29, 2025
Merged

[Devfile#1721] Adding renovate.json and test script to ensure sample can still be parsed and used by consumers for go sample#45
JslYoon merged 4 commits into
devfile-samples:mainfrom
JslYoon:devfile#1721

Conversation

@JslYoon

@JslYoon JslYoon commented Jul 16, 2025

Copy link
Copy Markdown
Contributor

[Devfile#1721] Adding renovate.json and test script to ensure sample can still be parsed and used by consumers
devfile/api#1721
Adding renovate.json and test script to ensure sample can still be parsed and used by consumers.

tests include:

odov3
non-terminating
validate-devfile-schemas

@thepetk thepetk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this one better to be on hold until Michael's comment is resolved

devfile-samples/devfile-sample-python-basic#70 (review)

Most probably we need to follow the same logic here too.

@thepetk thepetk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A quick comment for the usage of stackDir

Comment thread .github/workflows/validate-with-registry.yaml Outdated
@JslYoon JslYoon requested a review from thepetk July 25, 2025 16:20

@michael-valdron michael-valdron left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@JslYoon Can review once same changes as devfile/api#1718 are applied here.

Signed-off-by: Lucas Yoon <lyoon@lyoon-thinkpadp1gen7.boston.csb>
Comment thread renovate.json Outdated
Comment thread renovate.json
Comment thread tests/README.md Outdated
Comment thread .github/workflows/ci.yaml Outdated
Comment thread .github/workflows/ci.yaml
Signed-off-by: Lucas Yoon <lyoon@lyoon-thinkpadp1gen7.boston.csb>
@JslYoon JslYoon requested a review from michael-valdron July 28, 2025 15:48
Comment thread .github/workflows/ci.yaml Outdated
Co-authored-by: Michael Valdron <michael.valdron@gmail.com>
@JslYoon JslYoon requested a review from michael-valdron July 28, 2025 19:14
Comment thread renovate.json
Co-authored-by: Michael Valdron <michael.valdron@gmail.com>
@JslYoon JslYoon requested a review from michael-valdron July 29, 2025 17:57

@michael-valdron michael-valdron left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/lgtm

@thepetk thepetk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/lgtm

@JslYoon JslYoon merged commit 463f1ec into devfile-samples:main Jul 29, 2025
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