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

Release PR #1248

Merged
merged 5 commits into from
Aug 19, 2024
Merged

Release PR #1248

merged 5 commits into from
Aug 19, 2024

Conversation

karan-batavia
Copy link
Contributor

  • broken: work for enhanced derived source tagger

  • add edge between derived and original source for js

  • add original source edge for all languages

  • add tests

  • change inNode to astNode

  • refactor tests and add edge back to derived source

  • refactor DataflowExporterValidator.scala

  • scalafmt

  • add original source during export

  • remove existing derived source logic and refactor and add tests

  • scalafmt

  • address review comments

  • scalafmt

karan-batavia and others added 4 commits August 14, 2024 16:43
* broken: work for enhanced derived source tagger

* add edge between derived and original source for js

* add original source edge for all languages

* add tests

* change `inNode` to `astNode`

* refactor tests and add edge back to derived source

* refactor DataflowExporterValidator.scala

* scalafmt

* add original source during export

* remove existing derived source logic and refactor and add tests

* scalafmt

* address review comments

* scalafmt
* Dynamic rule merging logic

* added dynamic unit test

* merging logic change
add logs and handle exceptions for inference api endpoint tagger
pandurangpatil
pandurangpatil previously approved these changes Aug 19, 2024
* refactor language module

* more refactor

* refactor in test cases

* remove deprecated module
Copy link
Member

@khemrajrathore khemrajrathore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@karan-batavia karan-batavia merged commit 198a01a into main Aug 19, 2024
15 checks passed
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.

4 participants