Skip to content

Global call graph test#1018

Merged
mpharrigan merged 4 commits intoquantumlib:mainfrom
mpharrigan:2024-05/all-call-graph-test
Aug 22, 2024
Merged

Global call graph test#1018
mpharrigan merged 4 commits intoquantumlib:mainfrom
mpharrigan:2024-05/all-call-graph-test

Conversation

@mpharrigan
Copy link
Copy Markdown
Collaborator

@mpharrigan mpharrigan commented May 29, 2024

This PR has two commits

First, I add a dev_tool module for making one big call graph. This can be a useful starting point for querying other properties and idiosyncrasies of the bloq library; but here I use it to add a test that all bloq examples have a valid call graph.

Then, I make two minor fixes revealed by running the "all call graph" test.

@mpharrigan mpharrigan force-pushed the 2024-05/all-call-graph-test branch from a100d08 to 9ffe64b Compare August 22, 2024 20:53
@mpharrigan mpharrigan marked this pull request as ready for review August 22, 2024 20:55
@mpharrigan
Copy link
Copy Markdown
Collaborator Author

I brought this up to speed. @tanujkhattar if you could PTAL I think this would be a decent slow test; and it's generally useful to have a script example for making a unified call graph

@mpharrigan mpharrigan enabled auto-merge (squash) August 22, 2024 22:49
@mpharrigan mpharrigan merged commit 0d9d974 into quantumlib:main Aug 22, 2024
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