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

[FEATURE-REQUEST]: Creating Action Library for diff-match-patch #1064

Open
wants to merge 21 commits into
base: main
Choose a base branch
from

Conversation

twschutz
Copy link
Contributor

  1. Adds action library for diff-match-patch supporting all public functions described in the API. Functions are defined in diff.py, match.py, and patch.py respectively.
  2. Adds series of tests to check correct behavior of defined functions. Tests are created in tests/test_diff.py, tests/test_match.py, and tests/test_patch.py.
  3. Updates jac_misc README.md and creates a new README.md in jac_misc/dmp to document functions and provide helpful examples.

@twschutz twschutz changed the title [FEATURE-REQUEST]: Creating Action Library for diff-match-patch [WIP][FEATURE-REQUEST]: Creating Action Library for diff-match-patch Apr 18, 2023
@twschutz twschutz changed the title [WIP][FEATURE-REQUEST]: Creating Action Library for diff-match-patch [FEATURE-REQUEST]: Creating Action Library for diff-match-patch Apr 21, 2023
@twschutz twschutz marked this pull request as ready for review April 21, 2023 17:09
@sonarcloud
Copy link

sonarcloud bot commented Apr 28, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@ypkang ypkang self-requested a review May 1, 2023 15:35
@ypkang
Copy link
Contributor

ypkang commented Aug 30, 2023

@twschutz is this feature complete and ready for merge?

@twschutz
Copy link
Contributor Author

twschutz commented Sep 1, 2023 via email

@ypkang ypkang removed their request for review April 23, 2024 19:02
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.

None yet

2 participants