[HMA] Remove IContentTypeConfigStore from HMA#1707
[HMA] Remove IContentTypeConfigStore from HMA#1707haianhng31 wants to merge 6 commits intofacebook:mainfrom
Conversation
Dcallies
left a comment
There was a problem hiding this comment.
From the CI output, it looks like it's importing threatexchange 1.1.1
This is because our 1.2 version publish failed!
https://github.com/facebook/ThreatExchange/actions/runs/12247017442/job/34164057430
We'll have to fix that before this works
|
|
||
| import flask | ||
|
|
||
| from threatexchange.cli.storage.interfaces import IContentTypeConfigStore |
There was a problem hiding this comment.
I think you are getting a mypy failure here because this might need a missing __init.py__ file in the interfaces directory
There was a problem hiding this comment.
i don't know why mypy keeps complaining about not finding the module. I have checked:
- threatexchange package is installed
threatexchange.storage.interfacesis a valid module- has
__init__.pyin interfaces directory
|
Okay, I think I fixed the build and package, try re-submitting to see if everything passes again. |
|
You have the same problem I fixed in #1718 - the You can confirm that the folder is included by running |
| @@ -0,0 +1 @@ | |||
| # Copyright (c) Meta Platforms, Inc. and affiliates. | |||
There was a problem hiding this comment.
You'll need to move this to its own PR, then we'll need to bump the version and release a new bugfix version!
Summary