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

Conversation

Samyak2
Copy link
Contributor

@Samyak2 Samyak2 commented Jul 12, 2022

Changes

  • Formatting (can be ignored for review): 09b7448
  • Fix type errors and add missing type annotations: e970130
  • Use local logger instead of global: 24e497d
  • Add docstrings: bbdc52d
  • Minor unbound variables error: ee808de

@Samyak2 Samyak2 added this to the v0.10.0 milestone Jul 12, 2022
@Samyak2 Samyak2 requested a review from kartikay-bagla July 12, 2022 07:19
@gitpod-io
Copy link

gitpod-io bot commented Jul 12, 2022

@netlify
Copy link

netlify bot commented Jul 12, 2022

Deploy Preview for frontend-sb canceled.

Name Link
🔨 Latest commit ee808de
🔍 Latest deploy log https://app.netlify.com/sites/frontend-sb/deploys/62cd209cbeb3e40009e5787a

Copy link
Contributor

@kartikay-bagla kartikay-bagla left a comment

Choose a reason for hiding this comment

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

LGTM

@Samyak2 Samyak2 merged commit 6e2afdf into develop Jul 13, 2022
@Samyak2 Samyak2 deleted the refactor/anomaly-data-view-lints branch July 13, 2022 03:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants