new version of verification process,date assignment problem solved#217
new version of verification process,date assignment problem solved#217AdrianCabreraPhi wants to merge 17 commits intomasterfrom
Conversation
|
Hello @AdrianCabreraPhi! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2022-04-22 06:54:23 UTC |
manuelpastor
left a comment
There was a problem hiding this comment.
The changes can break the execution if the library KnowledgeHubAPI and dot_env are not in the flame environment
I cannot merge into master unless:
- You include all the compulsory libraries in the Flame environment
- You import them from within the functions, ideally protecting the execution from a crash with a trial-except or checking the library presence
No description provided.