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

Normalize inline puml line ending before hashing #1174

Merged
merged 1 commit into from
Apr 3, 2020

Conversation

ang-zeyu
Copy link
Contributor

@ang-zeyu ang-zeyu commented Apr 1, 2020

What is the purpose of this pull request? (put "X" next to an item, remove the rest)

• [x] Bug fix

What is the rationale for this request?
To normalize the line endings in inline puml content used to get the file name of the puml image so that it is consistent across windows / unix systems

What changes did you make? (Give an overview)

const normalizedContent = content.replace(/\r\n/g, '\n');
const hashedContent = cryptoJS.MD5(normalizedContent).toString();

Is there anything you'd like reviewers to focus on?
na

Testing instructions:
na

Proposed commit message: (wrap lines at 72 characters)
Normalize inline puml line ending before hashing

@ang-zeyu
Copy link
Contributor Author

ang-zeyu commented Apr 1, 2020

@crphang could I get your review here? Can't seem to tag you 😅

@crphang
Copy link
Contributor

crphang commented Apr 2, 2020

Didn't get the ping. LGTM!

@ang-zeyu ang-zeyu added this to the v2.13.2 milestone Apr 2, 2020
@ang-zeyu ang-zeyu merged commit fc51d0f into MarkBind:master Apr 3, 2020
Tejas2805 added a commit to Tejas2805/markbind that referenced this pull request Apr 9, 2020
* 'master' of https://github.com/MarkBind/markbind: (41 commits)
  Document adding new site content in DG (MarkBind#1153)
  Add relative date feature (MarkBind#908)
  Use <br> to separate lines of code block (MarkBind#1176)
  Parse popovers for footnotes (MarkBind#1155)
  Resolve comments
  Expand test extensions and fix whitespace checks (MarkBind#1156)
  Address comments
  Upgrade js-beautify and provide option to turn it off (MarkBind#1116)
  Normalize inline puml line ending before hashing (MarkBind#1174)
  Update tests (MarkBind#1178)
  Remove fixed bugs from test\functional\test_site\bugs\index.md` (MarkBind#1148)
  Fix bug in Search for UG and DG (MarkBind#1147)
  Add inline puml support (MarkBind#1100)
  Fix hrefs for headings with angular brackets (MarkBind#1089)
  Update tests for 2.13.1 (MarkBind#1169)
  2.13.1
  Update vue-strap version to v2.0.1-markbind.39
  Fix fontawsome icons don't show underlines to indicate modal/tooltip (MarkBind#1133)
  2.13.0
  Update test files
  ...
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.

3 participants