Skip to content

Conversation

JslYoon
Copy link
Contributor

@JslYoon JslYoon commented Jul 14, 2025

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

tests include:

validate-devfile-schemas
check non terminating
python dependencies, lint, and pytests (if exists)

@JslYoon
Copy link
Contributor Author

JslYoon commented Jul 14, 2025

I did make this PR, I am not sure variables such as ${{ env.MINIKUBE_VERSION }} or ${{ env.KUBERNETES_VERSION }} is declared

Copy link
Contributor

@michael-valdron michael-valdron left a comment

Choose a reason for hiding this comment

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

@JslYoon Instead of trying to duplicate, I would suggest creating a new workflow that first pulls the devfile/registry repository, then runs the sample against each test suite. Also, good to create new check_*.sh scripts as the sample itself is the only input, the ones from devfile/registry look for multiple stacks under a stacks/ directory.

@JslYoon JslYoon force-pushed the devfile#1718-renovate branch 3 times, most recently from 3629a8a to 0f2b1a2 Compare July 16, 2025 18:42
@JslYoon JslYoon requested a review from michael-valdron July 16, 2025 18:45
Copy link
Contributor

@thepetk thepetk left a comment

Choose a reason for hiding this comment

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

Just a small comment on the script usage

@JslYoon JslYoon force-pushed the devfile#1718-renovate branch from 6180598 to 2772b8a Compare July 24, 2025 19:33
@JslYoon JslYoon requested a review from thepetk July 24, 2025 19:39
@JslYoon JslYoon self-assigned this Jul 24, 2025
@JslYoon JslYoon added the enhancement New feature or request label Jul 24, 2025
Signed-off-by: Lucas Yoon <lyoon@lyoon-thinkpadp1gen7.boston.csb>
@JslYoon JslYoon force-pushed the devfile#1718-renovate branch from 2772b8a to 2fe8e1e Compare July 25, 2025 14:31
Signed-off-by: Lucas Yoon <lyoon@lyoon-thinkpadp1gen7.boston.csb>
@JslYoon JslYoon force-pushed the devfile#1718-renovate branch from eac1307 to 3ab30f3 Compare July 25, 2025 14:37
@JslYoon JslYoon requested a review from michael-valdron July 25, 2025 14:38
Copy link
Contributor

@michael-valdron michael-valdron left a comment

Choose a reason for hiding this comment

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

/lgtm

Copy link
Contributor

@michael-valdron michael-valdron left a comment

Choose a reason for hiding this comment

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

@JslYoon Let's takeout the odo variable

Co-authored-by: Michael Valdron <michael.valdron@gmail.com>
@JslYoon JslYoon requested a review from michael-valdron July 28, 2025 14:26
Copy link
Contributor

@michael-valdron michael-valdron left a comment

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 2057a76 into devfile-samples:main Jul 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants