Correcting Integer conversion type - Githubissues

chef / automate

Chef Automate provides a full suite of enterprise capabilities for maintaining continuous visibility into application, infrastructure, and security automation.

https://automate.chef.io/

Apache License 2.0

227 stars 113 forks source link

Correcting Integer conversion type #8636

Closed swatiganesh closed 2 weeks ago

swatiganesh commented 1 month ago

:nut_and_bolt: Description: What code changed, and why?

Correcting Integer conversion type

:chains: Related Resources

https://progresssoftware.atlassian.net/browse/CHEF-12180

:+1: Definition of Done

Correcting Integer conversion type

:athletic_shoe: How to Build and Test the Change

rebuild components/automate-deployment/ rebuild components/automate-cli/ rebuild components/config-mgmt-service/

:white_check_mark: Checklist

All PRs must tick these:

[x] I have read the CONTRIBUTING document.
[x] All commits signed-off for the Developer Certification of Origin.

With occasional exceptions, all PRs from Progress employees must tick these:

[ ] Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
[ ] Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
[ ] Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
[ ] Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
[ ] Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

[ ] Tests added/updated? (all new code needs new tests)
[ ] Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.

:camera: Screenshots, if applicable

netlify[bot] commented 1 month ago

Deploy Preview for chef-automate processing.

Name	Link
Latest commit	25ced419c2680cf62431aed6dcb03fa208090f8d
Latest deploy log	https://app.netlify.com/sites/chef-automate/deploys/6729bd9ba3b65600083b2390

sonarqube-for-infrastructure-prod[bot] commented 1 month ago

Quality Gate failed

Failed conditions
2 New issues
23.4% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube

Catch issues before they fail your Quality Gate with our IDE extension SonarLint