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

Update dependency #3

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update dependency #3

wants to merge 2 commits into from

Conversation

aoli-al
Copy link

@aoli-al aoli-al commented Apr 3, 2023

Use the upstream soot-infoflow from maven repository.

@khaled-e-a
Copy link
Collaborator

Hi, thank you fore helping improve Slicer4J!
May I ask what the purpose of this pull request? and what regression testing you did to check it is not breaking the tool?
Thank you.

@aoli-al
Copy link
Author

aoli-al commented Apr 4, 2023

Hi! I'm trying to use Slicer4j to analyze jena https://github.com/apache/jena. I got errors while instrumenting the application. The errors disappear after I upgrade the dependency.

I don't have any integration test available right now but all unit tests passed on my local machine.

@khaled-e-a
Copy link
Collaborator

Thank you for letting me know, we are currently updating the tool and will test and accept your pull request once we are done updating it. Thank you!

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