chore(sequence): remove endpoint comparison helpers

[connorskees]

Remove redundant endpoint comparison functions on interval helpers, compareStarts and compareEnds. We could reduce code churn by keeping these functions around and rewriting them in terms of the methods on IInterval, but I think just removing them entirely is the nicer solution. The code churn should be minimal.

Question for reviewers: should we deprecate these functions in main?

[Abe27342]

This looks good. We should deprecate in main first since it'll merge conflict with this and add a changeset to that PR and this PR (just point people at compareStart/compareEnd as you have)

[connorskees]

Deprecation added in #17127

[msfluid-bot]

⯆ @fluid-example/bundle-size-tests: -190 Bytes

Metric Name	Baseline Size	Compare Size	Size Diff
aqueduct.js	442.97 KB	442.88 KB	⯆ -96 Bytes
connectionState.js	680 Bytes	680 Bytes	■ No change
containerRuntime.js	237 KB	237 KB	■ No change
loader.js	147.92 KB	147.92 KB	■ No change
map.js	46.4 KB	46.4 KB	■ No change
matrix.js	140.13 KB	140.13 KB	■ No change
odspDriver.js	89.11 KB	89.11 KB	■ No change
odspPrefetchSnapshot.js	41.78 KB	41.78 KB	■ No change
sharedString.js	160.51 KB	160.42 KB	⯆ -94 Bytes
sharedTree2.js	240.31 KB	240.31 KB	■ No change
Total Size	1.65 MB	1.65 MB	⯆ -190 Bytes

---

Baseline commit: 4c3325d

Generated by 🚫 dangerJS against c2ba8db

[github-actions]

This PR is ready to merge! Please review it and squash merge into next: @sonalideshpandemsft @tylerbutler @scottn12