Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Integration Test framework and an initial test for Document Code #1291
Integration Test framework and an initial test for Document Code #1291
Changes from 52 commits
b451326
b5d19c9
ce239d6
4d84e12
dacdd76
ccfee5b
932b1d2
7dc4b37
cfac44d
bac793b
ded891e
4edd3a1
5bc49ba
189dd04
fdff59f
f2e011c
5efd0c5
1a946f7
9eaa635
8cd8ffd
16df516
f8b8235
a7243dd
9665434
829e6ab
c1a0aba
288eeb4
766b463
27f4b5e
6d30b56
f32ec5a
ae67ae2
9778ff2
78de66c
17df15e
1a6793a
366e3a3
05d7f4a
e1653c3
842d42d
ab26297
131cb36
b2252b7
b5b2812
f9697e6
4dd4bf5
e0c1bd1
da9dbb9
374dfec
926dc2b
64a501c
764dbf9
e763e34
14d07b9
62c8d29
7eeb1cd
3f45fc1
c5c2744
4d61876
940f5c3
fe0865f
8636991
1ebab91
7912614
f65fc55
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
There's a script in sourcegraph/cody, agent/scripts/export-cody-http-recording-tokens.sh . Can we standardize on using those tokens? Everyone has access to them and there are different variants for rate limited tokens, etc. if we need them down the road.
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.
...weird!
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.
Can you elaborate?
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.
or let's conditionally print an error or something.
Ditto below.
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.
noob question from me about Kotlin versioning... IntelliJ Platform also has some Kotlin pieces. What's the versioning constraints between our build and the IntelliJ hosts?
According to https://plugins.jetbrains.com/docs/intellij/using-kotlin.html#kotlin-standard-library , 2024.1 bundles 1.9.22. But I noticed "thing" that is our debug target, Main/Duke Icon, is 2022.something.
Does this mean we need to
kotlin.stdlib.default.dependency=true
? What does that mean if we're using IntelliJ's kotlin-isms (I was just looking at using their Kotlin Coroutine integration for the background task autocompleting repo names...)