Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: print clang-format output #186

Merged
merged 26 commits into from
Jul 17, 2018
Merged
Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
3bdc619
Started working using regex
lee-tammy Jul 8, 2018
ab8bc97
Working on apply patches in jsdiff
lee-tammy Jul 9, 2018
0fa2476
Able to separate string into multiple diffs and print them
lee-tammy Jul 9, 2018
314e3ca
Added comments and changed formatting -- Has special character bug
lee-tammy Jul 9, 2018
d692f24
Changed output formatting -- does not support utf8 characters
lee-tammy Jul 10, 2018
7f34080
Revert changes in a comment
lee-tammy Jul 10, 2018
bb5f697
Changed original substring function to special substring that counts …
lee-tammy Jul 10, 2018
22933d6
Added dependencies
lee-tammy Jul 10, 2018
36d0299
Made some changes
lee-tammy Jul 11, 2018
13a78ad
Split functions and added comments -- possible error when directory i…
lee-tammy Jul 11, 2018
ff94ab0
Forgot to add diff dependency
lee-tammy Jul 11, 2018
e4d1945
Added some comments and added some error checks
lee-tammy Jul 12, 2018
5c6f5f5
Updated regex -- should work on node 6 now
lee-tammy Jul 12, 2018
7844b05
Added a test for formating
lee-tammy Jul 13, 2018
b46b96e
Commit for merge
lee-tammy Jul 13, 2018
b7f7c31
Merged conflicts
lee-tammy Jul 13, 2018
850a455
Added tests and made minor changes
lee-tammy Jul 14, 2018
9b4521a
Fixed comment
lee-tammy Jul 16, 2018
8f02bce
Used lint and format
lee-tammy Jul 16, 2018
a37c01b
Changed existing tests
lee-tammy Jul 16, 2018
ddd5d81
Removed Logger import statement b/c it is not used anymore
lee-tammy Jul 16, 2018
d8575d4
Fixed tests and added one to check for error
lee-tammy Jul 17, 2018
bc47938
Fixed line over 80 chars
lee-tammy Jul 17, 2018
b57c458
forgot to run lint
lee-tammy Jul 17, 2018
69b3211
Merge branch 'master' into print-format-lines-v3
kjin Jul 17, 2018
2b80caa
Merge branch 'master' into print-format-lines-v3
kjin Jul 17, 2018
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fixed comment
  • Loading branch information
lee-tammy committed Jul 16, 2018
commit 9b4521a3edcb32625905b7ec9d8ec211d65e4e6b
6 changes: 3 additions & 3 deletions src/format.ts
Original file line number Diff line number Diff line change
@@ -159,9 +159,9 @@ function getReplacements(fileXML: string): Replacement[] {
xmlLines = xmlLines.slice(1, xmlLines.length - 1);

for (let i = 0; i < xmlLines.length; i++) {
// Uses regex to capture the xml attribute and element
// XML format: <replacement offset='OFFSET'
// length='LENGTH'>FIX</replacement>
// Uses regex to capture the xml attributes and element
// XML format:
// <replacement offset='OFFSET' length='LENGTH'>FIX</replacement>
const offset: string[]|null = (/offset=\'(\d+)\'/g).exec(xmlLines[i]);
const length: string[]|null = (/length=\'(\d+)\'/g).exec(xmlLines[i]);
const fix: string[]|null =