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

Bump diff and codelyzer #27

Closed
wants to merge 1 commit into from

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Jul 18, 2024

Bumps diff to 4.0.2 and updates ancestor dependency codelyzer. These dependencies need to be updated together.

Updates diff from 2.2.3 to 4.0.2

Changelog

Sourced from diff's changelog.

Release Notes

Future breaking 6.0.0 release

Commits

  • #497 diffWords behavior has been radically changed. Previously, even with ignoreWhitespace: true, runs of whitespace were tokens, which led to unhelpful and unintuitive diffing behavior in typical texts. Specifically, even when two texts contained overlapping passages, diffWords would sometimes choose to delete all the words from the old text and insert them anew in their new positions in order to avoid having to delete or insert whitespace tokens. Whitespace sequences are no longer tokens as of this release, which affects both the generated diffs and the counts.

    Runs of whitespace are still tokens in diffWordsWithSpace.

    As part of the changes to diffWords, a new .postProcess method has been added on the base Diff type, which can be overridden in custom Diff implementations.

    diffLines with ignoreWhitespace: true will no longer ignore the insertion or deletion of entire extra lines of whitespace at the end of the text. Previously, these would not show up as insertions or deletions, as a side effect of a hack in the base diffing algorithm meant to help ignore whitespace in diffWords. More generally, the undocumented special handling in the core algorithm for ignored terminals has been removed entirely. (This special case behavior used to rewrite the final two change objects in a scenario where the final change object was an addition or deletion and its value was treated as equal to the empty string when compared using the diff object's .equals method.)

  • #500 diffChars now diffs Unicode code points instead of UTF-16 code units.

  • #508 parsePatch now always runs in what was previously "strict" mode; the undocumented strict option has been removed. Previously, by default, parsePatch (and other patch functions that use it under the hood to parse patches) would accept a patch where the line counts in the headers were inconsistent with the actual patch content - e.g. where a hunk started with the header @@ -1,3 +1,6 @@, indicating that the content below spanned 3 lines in the old file and 6 lines in the new file, but then the actual content below the header consisted of some different number of lines, say 10 lines of context, 5 deletions, and 1 insertion. Actually trying to work with these patches using applyPatch or merge, however, would produce incorrect results instead of just ignoring the incorrect headers, making this "feature" more of a trap than something actually useful. It's been ripped out, and now we are always "strict" and will reject patches where the line counts in the headers aren't consistent with the actual patch content.

  • #435 Fix parsePatch handling of control characters. parsePatch used to interpret various unusual control characters - namely vertical tabs, form feeds, lone carriage returns without a line feed, and EBCDIC NELs - as line breaks when parsing a patch file. This was inconsistent with the behavior of both JsDiff's own diffLines method and also the Unix diff and patch utils, which all simply treat those control characters as ordinary characters. The result of this discrepancy was that some well-formed patches - produced either by diff or by JsDiff itself and handled properly by the patch util - would be wrongly parsed by parsePatch, with the effect that it would disregard the remainder of a hunk after encountering one of these control characters.

  • #439 Prefer diffs that order deletions before insertions. When faced with a choice between two diffs with an equal total edit distance, the Myers diff algorithm generally prefers one that does deletions before insertions rather than insertions before deletions. For instance, when diffing abcd against acbd, it will prefer a diff that says to delete the b and then insert a new b after the c, over a diff that says to insert a c before the b and then delete the existing c. JsDiff deviated from the published Myers algorithm in a way that led to it having the opposite preference in many cases, including that example. This is now fixed, meaning diffs output by JsDiff will more accurately reflect what the published Myers diff algorithm would output.

  • #455 The added and removed properties of change objects are now guaranteed to be set to a boolean value. (Previously, they would be set to undefined or omitted entirely instead of setting them to false.)

  • #464 Specifying {maxEditLength: 0} now sets a max edit length of 0 instead of no maximum.

  • #460 Added oneChangePerToken option.

  • #467 Consistent ordering of arguments to comparator(left, right). Values from the old array will now consistently be passed as the first argument (left) and values from the new array as the second argument (right). Previously this was almost (but not quite) always the other way round.

  • #480 Passing maxEditLength to createPatch & createTwoFilesPatch now works properly (i.e. returns undefined if the max edit distance is exceeded; previous behavior was to crash with a TypeError if the edit distance was exceeded).

  • #486 The ignoreWhitespace option of diffLines behaves more sensibly now. values in returned change objects now include leading/trailing whitespace even when ignoreWhitespace is used, just like how with ignoreCase the values still reflect the case of one of the original texts instead of being all-lowercase. ignoreWhitespace is also now compatible with newlineIsToken. Finally, diffTrimmedLines is deprecated (and removed from the docs) in favour of using diffLines with ignoreWhitespace: true; the two are, and always have been, equivalent.

  • #490 When calling diffing functions in async mode by passing a callback option, the diff result will now be passed as the first argument to the callback instead of the second. (Previously, the first argument was never used at all and would always have value undefined.)

  • #489 this.options no longer exists on Diff objects. Instead, options is now passed as an argument to methods that rely on options, like equals(left, right, options). This fixes a race condition in async mode, where diffing behaviour could be changed mid-execution if a concurrent usage of the same Diff instances overwrote its options.

  • #518 linedelimiters no longer exists on patch objects; instead, when a patch with Windows-style CRLF line endings is parsed, the lines in lines will end with \r. There is now a new autoConvertLineEndings option, on by default, which makes it so that when a patch with Windows-style line endings is applied to a source file with Unix style line endings, the patch gets autoconverted to use Unix-style line endings, and when a patch with Unix-style line endings is applied to a source file with Windows-style line endings, it gets autoconverted to use Windows-style line endings.

  • #521 **the callback option is now supported by structuredPatch, `createPatch

  • #529 parsePatch can now parse patches where lines starting with -- or ++ are deleted/inserted; previously, there were edge cases where the parser would choke on valid patches or give wrong results.

  • #530 Added ignoreNewlineAtEof optiontodiffLines`

v5.2.0

Commits

  • #411 Big performance improvement. Previously an O(n) array-copying operation inside the innermost loop of jsdiff's base diffing code increased the overall worst-case time complexity of computing a diff from O(n²) to O(n³). This is now fixed, bringing the worst-case time complexity down to what it theoretically should be for a Myers diff implementation.
  • #448 Performance improvement. Diagonals whose furthest-reaching D-path would go off the edge of the edit graph are now skipped, rather than being pointlessly considered as called for by the original Myers diff algorithm. This dramatically speeds up computing diffs where the new text just appends or truncates content at the end of the old text.
  • #351 Importing from the lib folder - e.g. require("diff/lib/diff/word.js") - will work again now. This had been broken for users on the latest version of Node since Node 17.5.0, which changed how Node interprets the exports property in jsdiff's package.json file.
  • #344 diffLines, createTwoFilesPatch, and other patch-creation methods now take an optional stripTrailingCr: true option which causes Windows-style \r\n line endings to be replaced with Unix-style \n line endings before calculating the diff, just like GNU diff's --strip-trailing-cr flag.
  • #451 Added diff.formatPatch.
  • #450 Added diff.reversePatch.
  • #478 Added timeout option.

v5.1.0

  • #365 Allow early termination to limit execution time with degenerate cases

Commits

v5.0.0

... (truncated)

Commits

Updates codelyzer from 0.0.28 to 6.0.2

Release notes

Sourced from codelyzer's releases.

5.1.0

Bug Fixes

  • rule: 'contextual-decorator' - decorators with arguments, accessors and some missing decorators not being handled (#798) (efbcb69)
  • rule: template-banana-in-box regex failing in some environments (#820) (3b82574)

Features

  • rule: add use-injectable-provided-in (#814) (656816f)

5.0.1

Bug Fixes

  • rule: detect trackBy function when ngFor spans multiple lines (#813) (2260dfd)
  • rule: template-accessibility-label-for not recognizing options and interpolated values (#812) (1fb5d8a)

5.0.0

🙇🏼‍♂️ Thanks to @​rafaelss95, @​mohammedzamakhan, @​wKoza, @​mgechev

BREAKING CHANGES

Angular CLI will automatically migrate your projects to reflect the latest config.

  • contextual-life-cycle is renamed to contextual-lifecycle
  • no-conflicting-life-cycle-hooks is renamed to no-conflicting-lifecycle
  • no-life-cycle-call is renamed to no-lifecycle-call
  • use-life-cycle-interface is renamed to use-lifecycle-interface
  • decorator-not-allowed is renamed to contextual-decorator
  • enforce-component-selector is renamed to use-component-selector
  • no-output-named-after-standard-event is renamed to no-output-native
  • use-host-property-decorator is renamed to no-host-metadata-property
  • use-input-property-decorator is renamed to no-inputs-metadata-property
  • use-output-property-decorator is renamed to no-outputs-metadata-property
  • no-queries-parameter is renamed to no-queries-metadata-property
  • pipe-impure is renamed to no-pipe-impure
  • use-view-encapsulation is renamed to use-component-view-encapsulation
  • i18n is renamed to template-i18n
  • banana-in-box is renamed to template-banana-in-box
  • no-template-call-expression is renamed to template-no-call-expression
  • templates-no-negated-async is renamed to template-no-negated-async
  • trackBy-function is renamed to template-use-track-by-function
  • no-attribute-parameter-decorator is renamed to no-attribute-decorator
  • max-inline-declarations is renamed to component-max-inline-declarations

Bug Fixes

  • rule: template-no-call-expression should allow $any usages (#735) (a75c204)
  • rule: no-input-rename reporting some failures incorrectly (#723) (f692dcb)
  • rule: prefer-inline-decorator should not set a max number of options (#787) (31b2b6a)
  • rule: add template-accessibility-elements-content to index.ts (#803) (1e8cbbf), closes #801

... (truncated)

Changelog

Sourced from codelyzer's changelog.

Changelog

All notable changes to this project will be documented in this file. See standard-version for commit guidelines.

6.0.1 (2020-09-30)

Bug Fixes

  • add Angular v11 in peer dependencies (#979) (cd88bd5)

6.0.0 (2020-09-30)

Bug Fixes

  • add Angular v11 in peer dependencies (#979) (cd88bd5)

5.3.0 (2020-07-02)

Features

  • add support for angular v10 (d27202c)

Bug Fixes

  • deps: update dependency app-root-path to v3 (#923) (4218091)
  • add support for tslint 6.0+ (#961) (ef5f158)
  • deps: update dependency codemirror to v5.51.0 (#914) (87bbfd5)
  • deps: update dependency typescript to v3 (#837) (49c2828)
  • 3.6 compile time errors (0066e93)

5.2.3 (2020-03-26)

Bug Fixes

  • add support for tslint 6.0+ (#961) (ef5f158)
  • deps: update dependency codemirror to v5.51.0 (#914) (87bbfd5)
  • deps: update dependency typescript to v3 (#837) (49c2828)
  • 3.6 compile time errors (0066e93)

5.2.1 (2020-03-03)

Bug Fixes

  • deps: update dependency codemirror to v5.51.0 (#914) (87bbfd5)
  • deps: update dependency typescript to v3 (#837) (49c2828)
  • 3.6 compile time errors (0066e93)

5.2.0 (2019-10-22)

Bug Fixes

... (truncated)

Commits

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
    You can disable automated security fix PRs for this repo from the Security Alerts page.

Bumps [diff](https://github.com/kpdecker/jsdiff) to 4.0.2 and updates ancestor dependency [codelyzer](https://github.com/mgechev/codelyzer). These dependencies need to be updated together.


Updates `diff` from 2.2.3 to 4.0.2
- [Changelog](https://github.com/kpdecker/jsdiff/blob/master/release-notes.md)
- [Commits](kpdecker/jsdiff@v2.2.3...v4.0.2)

Updates `codelyzer` from 0.0.28 to 6.0.2
- [Release notes](https://github.com/mgechev/codelyzer/releases)
- [Changelog](https://github.com/mgechev/codelyzer/blob/master/CHANGELOG.md)
- [Commits](https://github.com/mgechev/codelyzer/commits)

---
updated-dependencies:
- dependency-name: diff
  dependency-type: indirect
- dependency-name: codelyzer
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Jul 18, 2024
Copy link
Contributor Author

dependabot bot commented on behalf of github Jul 18, 2024

Looks like these dependencies are up-to-date now, so this is no longer needed.

@dependabot dependabot bot closed this Jul 18, 2024
@dependabot dependabot bot deleted the dependabot/npm_and_yarn/multi-d1d20093cc branch July 18, 2024 01:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants