Python: Add modeling of Flask blueprints #5078
Conversation
There was a problem hiding this comment.
I think this looks good, apart from a minor comment about a comment.
This will conflict with #5103, however, and as that PR already has a complicated commit history, I would like to avoid having to attempt to merge this PR into it.
Rather, I think an easier approach would be to simply recreate this PR using API graphs once the other PR has gone in (i.e. creating a new PR, cherry-picking the tests and change note, and redoing the implementation, which really shouldn't take long).
I'll be happy to do this. Certainly happier than I would be fixing merge conflicts.
python/ql/test/experimental/library-tests/frameworks/flask/routing_test.py
Outdated
Show resolved
Hide resolved
RasmusWL
force-pushed
the
flask-blueprints
branch
from
e6794fb to
bc8e613
Compare
|
I've already merged the API graphs PR, and have just fixed up this PR... Since I just spotted your review, I have also tried to reword that comment a bit. If it's still unclear, I'm happy to accept a suggestion making it better 👍 |
No description provided.