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

[873] Boomerang Update #232

Merged
merged 16 commits into from
Jul 19, 2022
Merged

[873] Boomerang Update #232

merged 16 commits into from
Jul 19, 2022

Conversation

turtledreams
Copy link
Contributor

@turtledreams turtledreams commented Jul 13, 2022

Currently async and sync both works (apm visible at request logs at dashboard) but apm functionality will be checked once more

@lgtm-com
Copy link

lgtm-com bot commented Jul 13, 2022

This pull request introduces 6 alerts and fixes 2 when merging ec4aa70 into a7d7e9f - view on LGTM.com

new alerts:

  • 6 for Unused variable, import, function or class

fixed alerts:

  • 2 for Duplicate property

@lgtm-com
Copy link

lgtm-com bot commented Jul 13, 2022

This pull request introduces 6 alerts and fixes 2 when merging 678ef28 into a7d7e9f - view on LGTM.com

new alerts:

  • 6 for Unused variable, import, function or class

fixed alerts:

  • 2 for Duplicate property

@lgtm-com
Copy link

lgtm-com bot commented Jul 14, 2022

This pull request introduces 6 alerts and fixes 2 when merging eb0a1bf into f633f1d - view on LGTM.com

new alerts:

  • 6 for Unused variable, import, function or class

fixed alerts:

  • 2 for Duplicate property

@turtledreams
Copy link
Contributor Author

image

@lgtm-com
Copy link

lgtm-com bot commented Jul 14, 2022

This pull request introduces 10 alerts and fixes 6 when merging 58449b2 into b1c0ebf - view on LGTM.com

new alerts:

  • 8 for Unused variable, import, function or class
  • 2 for Superfluous trailing arguments

fixed alerts:

  • 2 for Duplicate property
  • 2 for Superfluous trailing arguments
  • 2 for Unused variable, import, function or class

@turtledreams
Copy link
Contributor Author

so currently version of boomerang seems to be 1.0.0. Will investigate

@lgtm-com
Copy link

lgtm-com bot commented Jul 14, 2022

This pull request introduces 10 alerts and fixes 6 when merging 97de074 into b1c0ebf - view on LGTM.com

new alerts:

  • 8 for Unused variable, import, function or class
  • 2 for Superfluous trailing arguments

fixed alerts:

  • 2 for Duplicate property
  • 2 for Superfluous trailing arguments
  • 2 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Jul 15, 2022

This pull request introduces 10 alerts and fixes 6 when merging 617c78a into b1c0ebf - view on LGTM.com

new alerts:

  • 8 for Unused variable, import, function or class
  • 2 for Superfluous trailing arguments

fixed alerts:

  • 2 for Duplicate property
  • 2 for Superfluous trailing arguments
  • 2 for Unused variable, import, function or class

@lgtm-com
Copy link

lgtm-com bot commented Jul 19, 2022

This pull request introduces 6 alerts and fixes 2 when merging 9a58006 into b1c0ebf - view on LGTM.com

new alerts:

  • 6 for Unused variable, import, function or class

fixed alerts:

  • 2 for Duplicate property

@lgtm-com
Copy link

lgtm-com bot commented Jul 19, 2022

This pull request fixes 94 alerts when merging 6a4b038 into b1c0ebf - view on LGTM.com

fixed alerts:

  • 44 for Unused variable, import, function or class
  • 14 for Useless assignment to local variable
  • 10 for Superfluous trailing arguments
  • 8 for Useless conditional
  • 4 for Deleting non-property
  • 4 for Missing variable declaration
  • 4 for Incomplete string escaping or encoding
  • 2 for Duplicate property
  • 2 for Comparison between inconvertible types
  • 2 for Unneeded defensive code

@lgtm-com
Copy link

lgtm-com bot commented Jul 19, 2022

This pull request fixes 94 alerts when merging fb4cbc5 into b1c0ebf - view on LGTM.com

fixed alerts:

  • 44 for Unused variable, import, function or class
  • 14 for Useless assignment to local variable
  • 10 for Superfluous trailing arguments
  • 8 for Useless conditional
  • 4 for Deleting non-property
  • 4 for Missing variable declaration
  • 4 for Incomplete string escaping or encoding
  • 2 for Duplicate property
  • 2 for Comparison between inconvertible types
  • 2 for Unneeded defensive code

@ArtursKadikis ArtursKadikis merged commit 1fde3f3 into staging Jul 19, 2022
@ArtursKadikis ArtursKadikis deleted the 873 branch July 19, 2022 12:14
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