Skip to content

codeintel: Merge codeintellify

Warren Gifford requested to merge merge-codeintellify into main

Created by: chrismwendt

This merges the codeintellify repo into sourcegraph/sourcegraph, which makes it a lot easier to develop against, unifies the development environment and configs, etc.

According to this Sourcegraph search, there will be no impact on anyone else:

CleanShot 2021-11-24 at 13 38 49@2x

Because jsdom doesn't do layout, I had to disable this test:

    // jsdom does not support `scrollIntoView()` because it doesn't do layout.
    // https://github.com/jsdom/jsdom/issues/1695
    it.skip('hides the hover overlay when the hovered token intersects with a scrollBoundary', async () => {

And I mocked the HTMLElements and getBoundingClientRect() in overlayPosition.test.ts.

Original Slack thread

Merge request reports