-
-
Notifications
You must be signed in to change notification settings - Fork 51
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
feat: chordpro & chords over words compatibility #1004
feat: chordpro & chords over words compatibility #1004
Conversation
return [...emptyLines, ...lines, trailingLine]; | ||
} | ||
|
||
function pairChordsWithLyrics(chordsLine, lyricsLine) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Function pairChordsWithLyrics
has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.
} | ||
|
||
// eslint-disable-next-line no-unused-vars, @typescript-eslint/no-unused-vars | ||
function arrangeChordSheetLines(chordSheetLines: any[]) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Function arrangeChordSheetLines
has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.
return [...emptyLines, ...lines, trailingLine]; | ||
} | ||
|
||
function pairChordsWithLyrics(chordsLine, lyricsLine) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Function pairChordsWithLyrics
has 30 lines of code (exceeds 25 allowed). Consider refactoring.
… on master to text formatter
6569abb
to
d582eb6
Compare
Code Climate has analyzed commit d582eb6 and detected 0 issues on this pull request. View more on Code Climate. |
No description provided.