Skip to content

ci: Add recipe golden values#1647

Merged
thomasdhc merged 3 commits intomainfrom
donghyukc/golden_values
Apr 2, 2026
Merged

ci: Add recipe golden values#1647
thomasdhc merged 3 commits intomainfrom
donghyukc/golden_values

Conversation

@thomasdhc
Copy link
Copy Markdown
Contributor

What does this PR do ?

Add a one line overview of what this PR aims to accomplish.

Changelog

  • Add specific line by line info of high level changes in this PR.

Before your PR is "Ready for review"

Pre checks:

  • Make sure you read and followed Contributor guidelines
  • Did you write any new necessary tests?
  • Did you add or update any necessary documentation?

If you haven't finished some of the above items you can still open "Draft" PR.

Additional Information

  • Related to # (issue)

Signed-off-by: Dong Hyuk Chang <9426164+thomasdhc@users.noreply.github.com>
@thomasdhc thomasdhc requested a review from torsli April 1, 2026 18:24
Copy link
Copy Markdown
Contributor

@torsli torsli left a comment

Choose a reason for hiding this comment

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

Each golden value file is linked to a specific example. Is there a system in place to automatically update or deprecate the golden values if the system changes?

Some are blank and I'm not sure why. Otherwise, LGTM.

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.

This file is blank. Is that expected?

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.

This one too.

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.

Here too.

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.

?

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.

again

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.

just wanted to highlight all of them.

Signed-off-by: Dong Hyuk Chang <9426164+thomasdhc@users.noreply.github.com>
@thomasdhc thomasdhc merged commit f7b98a2 into main Apr 2, 2026
53 checks passed
@thomasdhc thomasdhc deleted the donghyukc/golden_values branch April 2, 2026 19:48
linnanwang pushed a commit that referenced this pull request Apr 24, 2026
* Add golden values

Signed-off-by: Dong Hyuk Chang <9426164+thomasdhc@users.noreply.github.com>

* Remove empty golden values

Signed-off-by: Dong Hyuk Chang <9426164+thomasdhc@users.noreply.github.com>

---------

Signed-off-by: Dong Hyuk Chang <9426164+thomasdhc@users.noreply.github.com>
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