Skip to content

Move action execution related code to campaigns package

Warren Gifford requested to merge campaigns-pkg into master

Created by: mrnugget

Opinions on this?

I think the code related to campaigns-specific functionality was sprawling and moving it to its own package makes API boundaries and dependencies clearer.

This is a first step that very stupidly capitalizes identifiers to export them and passes dependencies into the package (cfg.Endpoint, cfg.AccessToken). If we're happy with having a separate package, we can rename things (which we'll do anyway with the new flow) and possibly create other subpackages.

(Stacked on top of https://github.com/sourcegraph/src-cli/pull/229)

Merge request reports