Skip to content

Comments

Removing LGTM#874

Merged
gkorland merged 2 commits intoRedisJSON:masterfrom
chayim:ck-removing-deprecated-lgtm
Dec 20, 2022
Merged

Removing LGTM#874
gkorland merged 2 commits intoRedisJSON:masterfrom
chayim:ck-removing-deprecated-lgtm

Conversation

@chayim
Copy link
Contributor

@chayim chayim commented Nov 27, 2022

The LGTM service is being shut off in two weeks. This pull request aims to remove all references to LGTM. Perhaps LGTM usage should be replaced with codeql, or a repository preferred tool, but IMHO that's the point of a different pull request.

@CLAassistant
Copy link

CLAassistant commented Nov 27, 2022

CLA assistant check
All committers have signed the CLA.

@codecov
Copy link

codecov bot commented Nov 27, 2022

Codecov Report

Base: 81.41% // Head: 81.41% // No change to project coverage 👍

Coverage data is based on head (4352b28) compared to base (1153675).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #874   +/-   ##
=======================================
  Coverage   81.41%   81.41%           
=======================================
  Files          14       14           
  Lines        3879     3879           
=======================================
  Hits         3158     3158           
  Misses        721      721           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@gkorland gkorland merged commit 38bf9fc into RedisJSON:master Dec 20, 2022
rafie pushed a commit that referenced this pull request Dec 27, 2022
Co-authored-by: Guy Korland <gkorland@gmail.com>
(cherry picked from commit 38bf9fc)
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.

3 participants