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.

Sign up for GitHub

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

Jump to bottom

Add history of code intelligence #1628

Merged
merged 6 commits into from
Sep 23, 2020
Merged

Add history of code intelligence #1628

merged 6 commits into from
Sep 23, 2020

Conversation

aidaeology
Copy link
Contributor

From notes I took in various discussions.

@aidaeology
Add history of code intelligence
2343945 
From notes I took in various discussions.
@aidaeology aidaeology added this to the 3.21 milestone Sep 22, 2020
@aidaeology aidaeology requested review from nicksnyder and a team September 22, 2020 22:28
@aidaeology aidaeology self-assigned this Sep 22, 2020
@sourcegraph-bot
Copy link
Contributor

sourcegraph-bot commented Sep 22, 2020 •
edited
Loading

Notifying subscribers in CODENOTIFY files for diff b89f661...a970b44.

Notify File(s)
@nicksnyder handbook/engineering/code-intelligence/index.md
@sourcegraph/code-intel handbook/engineering/code-intelligence/index.md

@aidaeology
Copy link
Contributor Author

@sqs and @beyang I wrote a brief summary of code intelligence history based on my notes from previous discussions, @nicksnyder thought it would be good to add it here, please fact check for historical inaccuracies.

efritz
efritz approved these changes Sep 22, 2020
View reviewed changes
Copy link
Contributor

@efritz efritz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I think this is good context for anyone joining the team.

@aidaeology aidaeology requested review from sqs and beyang September 22, 2020 22:32
@sourcegraph-bot sourcegraph-bot mentioned this pull request Sep 22, 2020
Closed
sqs
sqs approved these changes Sep 23, 2020
View reviewed changes
Copy link
Member

@sqs sqs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. Approving pending @beyang's recommended edits.

@aidaeology aidaeology merged commit e705228 into main Sep 23, 2020
@aidaeology aidaeology deleted the codeintel-docs-history branch September 23, 2020 20:37
@macraig
Copy link
Contributor

macraig commented Sep 24, 2020

Thanks for capturing this! It's super useful

@sourcegraph-bot sourcegraph-bot mentioned this pull request Sep 29, 2020
Closed
17 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@beyang beyang beyang left review comments

@efritz efritz efritz approved these changes

@nicksnyder nicksnyder nicksnyder approved these changes

@sqs sqs sqs approved these changes

Assignees

@aidaeology aidaeology

Labels
None yet
Projects
None yet
Milestone
3.21
Development

Successfully merging this pull request may close these issues.
7 participants
@aidaeology @sourcegraph-bot @macraig @sqs @efritz @nicksnyder @beyang