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.

Sign up for GitHub

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

Jump to bottom

Fix lgtm warning #1972

Merged

FSM1 merged 1 commit into dev from fix/fix-lgtm-warning

Feb 23, 2022

Collaborator

Tbaut commented Feb 23, 2022

Those 2 desktop are always false

thanks to https://lgtm.com/projects/g/ChainSafe/ui-monorepo/rev/pr-90eecae9ffe076b64c86746fab2013730159b526


          fix lgtm warning

2087bd7

Tbaut requested review from asnaith, tanmoyAtb and FSM1

February 23, 2022 18:34

render bot commented Feb 23, 2022

Your Render PR Server URL is https://chainsafe-components-stage-pr-1972.onrender.com.

Follow its progress at https://dashboard.render.com/static/srv-c8b7s7fho1kpmae4bpd0.

render bot commented Feb 23, 2022

Your Render PR Server URL is https://storage-ui-stage-pr-1972.onrender.com.

Follow its progress at https://dashboard.render.com/static/srv-c8b7s87ho1kpmae4bphg.

render bot commented Feb 23, 2022

Your Render PR Server URL is https://files-ui-stage-pr-1972.onrender.com.

Follow its progress at https://dashboard.render.com/static/srv-c8b7s8nho1kpmae4bpv0.

github-actions bot added the Type: Bug Fix label

lgtm-com bot commented Feb 23, 2022

This pull request fixes 2 alerts when merging 2087bd7 into bb666c5 - view on LGTM.com

fixed alerts:

2 for Useless conditional

asnaith approved these changes

View reviewed changes

FSM1 approved these changes

View reviewed changes

FSM1 merged commit ab5e976 into dev

FSM1 deleted the fix/fix-lgtm-warning branch

February 23, 2022 20:58

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

Labels