Skip to content

Conversation

@LawnGnome
Copy link
Contributor

This is the src-cli component of sourcegraph/sourcegraph#15822.

There's also a new feature flag in here to maintain backward compatibility with older Sourcegraph versions, at the cost of extra work when parsing the campaign spec. I think that's the right tradeoff.

@LawnGnome LawnGnome requested a review from a team November 21, 2020 02:42
Copy link
Contributor

@mrnugget mrnugget left a comment

Choose a reason for hiding this comment

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

Good stuff! Really neat :)

@chrispine
Copy link

This is awesome, thanks!

This is the src-cli component of sourcegraph/sourcegraph#15822.

There's also a new feature flag in here to maintain backward
compatibility with older Sourcegraph versions, at the cost of extra work
when parsing the campaign spec. I think that's the right tradeoff.
@LawnGnome LawnGnome force-pushed the aharvey/env-passthrough branch from 02ba17d to 3ed24a6 Compare November 23, 2020 21:55
@LawnGnome LawnGnome merged commit 242d54f into main Nov 24, 2020
@LawnGnome LawnGnome deleted the aharvey/env-passthrough branch November 24, 2020 06:03
scjohns pushed a commit that referenced this pull request Apr 24, 2023
This is the src-cli component of sourcegraph/sourcegraph#15822.

There's also a new feature flag in here to maintain backward
compatibility with older Sourcegraph versions, at the cost of extra work
when parsing the campaign spec. I think that's the right tradeoff.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants