Skip to content

Fix undefined variable#60

Closed
sairoutine wants to merge 557 commits intoklaudiosinani:masterfrom
sairoutine:master
Closed

Fix undefined variable#60
sairoutine wants to merge 557 commits intoklaudiosinani:masterfrom
sairoutine:master

Conversation

@sairoutine
Copy link
Copy Markdown

themes is not defined in getRandomTheme function.
Therefore, if keys.length is 0, an uncaught error would be occur.

Mario Sinani added 30 commits July 8, 2017 15:16
@sairoutine
Copy link
Copy Markdown
Author

sairoutine commented Jun 30, 2022

@klaussinani
Hi! It seems that travis-ci has some troubles and does not work.
Could you have any ideas to fix it?

@klaudiosinani klaudiosinani self-requested a review July 1, 2022 20:34
@klaudiosinani
Copy link
Copy Markdown
Owner

@sairoutine Thank you for taking the time to contribute! I will have a look into the CI and review the PR asap

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Sep 4, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants