New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Refactoring code in src/meta/config.js #36

Open

rkensouh wants to merge 5 commits into CMU-17313Q:f24 from rkensouh:fixing-config.js

rkensouh commented Sep 2, 2024

Fixing the function handleTypeConversion to decrease complexity

Reem Konsowa and others added 4 commits

September 1, 2024 14:59


          changing from else-if to if in function

01e9771


          fixing indentation mistakes

765a1e8


          added comments for function created

3d385d5


          fixing comment style

61dfeb5

coveralls commented Sep 2, 2024

Pull Request Test Coverage Report for Build 10662110709

Details

19 of 23 (82.61%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.001%) to 82.672%

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/meta/configs.js	19	23	82.61%

Totals
Change from base Build 10550029021:	0.001%
Covered Lines:	22326
Relevant Lines:	25585

💛 - Coveralls


          making log print work

e0c8824

sonarcloud bot commented Sep 4, 2024

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet