Skip to content

Remove ignores in patch_click#2573

Closed
nipunn1313 wants to merge 1 commit into
psf:mainfrom
nipunn1313:ignore
Closed

Remove ignores in patch_click#2573
nipunn1313 wants to merge 1 commit into
psf:mainfrom
nipunn1313:ignore

Conversation

@nipunn1313

Copy link
Copy Markdown
Contributor

May help unblock #2431

Checklist - did you ...

(none of these necessary)

  • Add a CHANGELOG entry if necessary?
  • Add / update tests if necessary?
  • Add new / update outdated documentation?
@ichard26 ichard26 added C: maintenance Related to project maintenance, e.g. CI, testing, policy changes, releases ci: skip news Pull requests that don't need a changelog entry. labels Oct 30, 2021
@ichard26 ichard26 self-assigned this Oct 30, 2021
@ichard26

Copy link
Copy Markdown
Collaborator

While I'm happy we have multiple workarounds for this messy issue I'll be going with the old workaround we've been using successfully. This cast hides the fact that this code doesn't type check right while the pre-existing one is more explicit that it's a temporary workaround.

thanks for investigating!

@ichard26 ichard26 closed this Oct 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C: maintenance Related to project maintenance, e.g. CI, testing, policy changes, releases ci: skip news Pull requests that don't need a changelog entry.

2 participants