Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Adds Action changes field as option vec of serde_json value #431

Merged
merged 7 commits into from
Mar 25, 2024

Conversation

gpeacock
Copy link
Collaborator

There are no Rust APIS for this yet.
But it allows the values to pass through via actions.
This is a minimal change to allow the new actions. changes field to be set via JSON and read back that way.
Adding detailed Rust APIs should wait until the spec for this is completed

There are no Rust APIS for this yet.
But it allows the values to pass through via actions.
@codecov-commenter
Copy link

codecov-commenter commented Mar 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.86%. Comparing base (e0b7fcd) to head (51230f3).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #431      +/-   ##
==========================================
+ Coverage   79.81%   79.86%   +0.05%     
==========================================
  Files          82       82              
  Lines       25401    25494      +93     
==========================================
+ Hits        20275    20362      +87     
- Misses       5126     5132       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@dkozma dkozma left a comment

Choose a reason for hiding this comment

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

Question about a comment, but LGTM!

Copy link
Collaborator

@mauricefisher64 mauricefisher64 left a comment

Choose a reason for hiding this comment

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

👍

@gpeacock gpeacock merged commit d5b2c20 into main Mar 25, 2024
22 checks passed
@gpeacock gpeacock deleted the gpeacock/actions_changes branch March 25, 2024 17:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants