Skip to content
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.

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

Refactoring code in src/install.js #13

Open
VallinZ opened this issue Jan 17, 2025 · 2 comments · May be fixed by #22
Open

Refactoring code in src/install.js #13

VallinZ opened this issue Jan 17, 2025 · 2 comments · May be fixed by #22
Assignees

Comments

@VallinZ
Copy link

VallinZ commented Jan 17, 2025

In the src/install.js, there is a need to refactor the function called checkCIFlag on line 125 to reduce the cognitive complexity from 16 to 15 allowed. There are too many regular if statements and nested if statements, which make the code hard to read. Reducing the cognitive complexity of this function will help maintain high readability and understanding of this code for future testings and modifications.

@VallinZ
Copy link
Author

VallinZ commented Jan 17, 2025

I would like to work on this please!

Copy link

Great! I assigned you (@VallinZ) to the issue. Have fun working on it!

@VallinZ VallinZ changed the title Refactoring code in src/install.js to reduce cognitive complexity Refactoring code in src/install.js Jan 18, 2025
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 a pull request may close this issue.

1 participant