Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

callgraph: make pre-push target pass with newer pandas versions #74

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

henrirosten
Copy link
Contributor

Signed-off-by: Henri Rosten [email protected]

- Read all callgraph csv fields as strings to match queries also in
  cases where datatypes might not match
- Fix warnings reported by newer pycodestyle versions
- Fix build error with travis on python 3.7, reference:
  https://travis-ci.community/t/build-error-for-python-3-7-on-two-different-projects/12895

Signed-off-by: Henri Rosten <[email protected]>
@henrirosten henrirosten requested a review from bulwahn November 20, 2022 09:25
@henrirosten
Copy link
Contributor Author

henrirosten commented Nov 20, 2022

@bulwahn : I wanted to try using the callgraph tool in a personal project, but noticed the query_callgraph.py no longer worked with latest pandas version. Also, noticed some other issues with travis builds, which are now also fixed with this PR. I'm not sure if https://github.com/elisa-tech/workgroups is maintained anymore, or if you are the maintainer but in case you are, could we merge this PR to make the query_callgraph.py usable again and fix current CI issues.

@pahmann
Copy link

pahmann commented Nov 24, 2022

@henrirosten thanks for providing the patch. I will check with @bulwahn to give also his approval to have a 4 eyes review on your patch.
I will also put the PR to the agenda of the TSC meeting next week. (Nov 30) to conclude on the patch.

Thanks for fixing the issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants