Skip to content

Conversation

@preetham
Copy link

Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

Rubocop (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@jhua-vmware
Copy link
Collaborator

Please fix the action run. You should be able to run the action after every push form now on.

@preetham
Copy link
Author

preetham commented Jul 18, 2025

@jhua-vmware Fixed the issues reported in the action run, please approve the action run for the changes.

@jhua-vmware
Copy link
Collaborator

Can you fix unit/smoke test in https://github.com/vmware/singleton/actions/runs/16378688905?

@preetham
Copy link
Author

@jhua-vmware Tests are fixed, please verify.

jhua-vmware
jhua-vmware previously approved these changes Jul 19, 2025
@jhua-vmware
Copy link
Collaborator

You need to set up gpg key to sign your commit, can you make one commit?

@codecov
Copy link

codecov bot commented Jul 19, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.87%. Comparing base (6004b5d) to head (e9e32ae).
Report is 1 commits behind head on g11n-ruby-client.

Additional details and impacted files
@@                Coverage Diff                @@
##           g11n-ruby-client    #3232   +/-   ##
=================================================
  Coverage             92.87%   92.87%           
=================================================
  Files                    23       23           
  Lines                   491      491           
=================================================
  Hits                    456      456           
  Misses                   35       35           
Flag Coverage Δ
g11n-ruby-client 92.87% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: Preetham Kamidi <preetham.kamidi@broadcom.com>

Update: Version

Signed-off-by: Preetham Kamidi <preetham.kamidi@broadcom.com>

Fix: Header pattern

Signed-off-by: Preetham Kamidi <preetham.kamidi@broadcom.com>

Fix: Smoke tests

Signed-off-by: Preetham Kamidi <preetham.kamidi@broadcom.com>

Fix: Unit tests

Signed-off-by: Preetham Kamidi <preetham.kamidi@broadcom.com>

Fix: Switch to safe_load

Signed-off-by: Preetham Kamidi <preetham.kamidi@broadcom.com>

Update: Switch to CodeQL v3

Signed-off-by: Preetham Kamidi <preetham.kamidi@broadcom.com>
@preetham
Copy link
Author

@jhua-vmware Signed the latest commit, please verify.

@preetham preetham requested a review from jhua-vmware July 19, 2025 03:26
@preetham
Copy link
Author

@jhua-vmware @sjiang-cb If the changes are good, can you please merge the PR?

@jhua-vmware jhua-vmware merged commit 1516336 into vmware:g11n-ruby-client Jul 24, 2025
17 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.

3 participants