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

[871]_2 Deep Scan medium and low, non boomerang fixes #233

Merged
merged 14 commits into from
Jul 14, 2022
Merged

Conversation

turtledreams
Copy link
Contributor

No description provided.

…essed without null check prior at line 246 .
…t because the false branch of the condition 'this.ignore_visitor' at line 353 has been taken.
…'ratingShown' is originated from the assignment 'ratingShown = false' at line 2845 .
…value of variable 'segments' is originated from the return value of 'truncateObject()' defined at line 4201 .
…ampaign_id' is originated from the assignment 'campaign_id = campaign_id || setGetValueInStorage(...' at line 1228 .
…surveyShown' is originated from the assignment 'surveyShown = false' at line 2761 .
…viewSegments' is originated from the return value of 'truncateObject()' defined at line 4201 .
…ampaign_id' is originated from the assignment 'campaign_id = campaign_id || setGetValueInStorage(...' at line 1228 .
…surveyShown' is originated from the assignment 'surveyShown = false' at line 2761 .
@lgtm-com
Copy link

lgtm-com bot commented Jul 14, 2022

This pull request fixes 5 alerts when merging 4ba2776 into f633f1d - view on LGTM.com

fixed alerts:

  • 4 for Useless conditional
  • 1 for Unneeded defensive code

@lgtm-com
Copy link

lgtm-com bot commented Jul 14, 2022

This pull request fixes 3 alerts when merging 4327fef into f633f1d - view on LGTM.com

fixed alerts:

  • 2 for Useless conditional
  • 1 for Unneeded defensive code

@ArtursKadikis ArtursKadikis merged commit b1c0ebf into staging Jul 14, 2022
@ArtursKadikis ArtursKadikis deleted the 871_2 branch July 14, 2022 12:29
@lgtm-com
Copy link

lgtm-com bot commented Jul 14, 2022

This pull request fixes 2 alerts when merging a6548d9 into f633f1d - view on LGTM.com

fixed alerts:

  • 1 for Useless conditional
  • 1 for Unneeded defensive code

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.

2 participants