Skip to content
This repository has been archived by the owner on Sep 19, 2024. It is now read-only.

Remove unused declaration #642

Merged
merged 7 commits into from
Sep 8, 2023
Merged

Conversation

0x4007
Copy link
Member

@0x4007 0x4007 commented Aug 20, 2023

I got sick of seeing this in every pull request review.

Screenshot 2023-08-20 at 20 43 16

@netlify
Copy link

netlify bot commented Aug 20, 2023

Deploy Preview for ubiquibot-staging ready!

Name Link
🔨 Latest commit c02596b
🔍 Latest deploy log https://app.netlify.com/sites/ubiquibot-staging/deploys/64fa81508b7bfc0008725172
😎 Deploy Preview https://deploy-preview-642--ubiquibot-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@0x4007 0x4007 enabled auto-merge August 20, 2023 11:45
@0x4007 0x4007 mentioned this pull request Aug 20, 2023
@Draeieg Draeieg self-requested a review August 26, 2023 02:55
auto-merge was automatically disabled August 26, 2023 15:45

Merge queue setting changed

@0x4007 0x4007 added ping and removed ping labels Aug 29, 2023
rndquu
rndquu previously approved these changes Sep 6, 2023
}
logger.info(`Creating a new wallet_table record done, { data: ${JSON.stringify(_data)} }`);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that warning is weird because _data is used here

@0xcodercrane 0xcodercrane merged commit 1f6e1be into ubiquity:development Sep 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants