Skip to content
This repository has been archived by the owner on Sep 24, 2024. It is now read-only.

added text differences #54

Merged
merged 4 commits into from
Feb 5, 2024
Merged

Conversation

nerdxio
Copy link
Contributor

@nerdxio nerdxio commented Feb 5, 2024

No description provided.

@codecov-commenter
Copy link

Codecov Report

Attention: 6 lines in your changes are missing coverage. Please review.

Comparison is base (7853cc7) 96.25% compared to head (e815ca7) 95.50%.

Files Patch % Lines
server/server.go 37.50% 4 Missing and 1 partial ⚠️
api/root.go 0.00% 1 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #54      +/-   ##
==========================================
- Coverage   96.25%   95.50%   -0.76%     
==========================================
  Files          24       26       +2     
  Lines         668      690      +22     
==========================================
+ Hits          643      659      +16     
- Misses         19       24       +5     
- Partials        6        7       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


import "github.com/sergi/go-diff/diffmatchpatch"

func CompareText(txt1, txt2 string) string {
Copy link
Owner

Choose a reason for hiding this comment

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

@nerdxio Could you add here the line compare flag as well?

server/server.go Outdated
}
func compareTextPage(app *fiber.App) {
app.Get(TextsPath, func(c fiber.Ctx) error {
return c.Render(Prefix+"text_diff", newMap(map[string]any{
Copy link
Owner

Choose a reason for hiding this comment

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

text instead of text_diff

Copy link
Owner

Choose a reason for hiding this comment

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

text instead of text_diff.html

api/text_diff.go Outdated
Copy link
Owner

Choose a reason for hiding this comment

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

text

@ahelmy ahelmy merged commit bdf3cda into ahelmy:main Feb 5, 2024
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants