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.

Sign up for GitHub

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

Jump to bottom

Php small fixes #43

Merged

egor-bogomolov merged 3 commits into main from php-small-fixes

Feb 6, 2023

Collaborator

max-martynov commented Feb 4, 2023

A small update on PHP mining:

Resolve this issue: Change plugin-utilities dependency declaration #40
Change the way of mining ReturnsTo edges. Before there used to be more edges than necessary, now we have an edge for every return statement and the edge from the last instruction (in case it's not a return statement). Add tests for clarity.
Fix the issue with spamming logs while running on CodeSearchNet dataset by passing "-Didea.is.internal=false" to JVM args.

max-martynov added 3 commits

February 4, 2023 22:23


          Update dependency on plugin-utilities-core

0b65eb1


          Change the way of mining ReturnsTo edges for PHP, add tests

f9be531


          Fix the issue with spamming logs generated inside PHP plugin

fea1827

max-martynov requested a review from egor-bogomolov

February 4, 2023 21:35

egor-bogomolov approved these changes

View reviewed changes

Collaborator

egor-bogomolov left a comment

LGTM, thanks!

egor-bogomolov merged commit 6b647ea into main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet