Skip to content

Conversation

@jdpleiness
Copy link
Contributor

Insights were automatically deleted, add option to keep insight created during validation via deleteWhenDone:

externalService:
  kind: GITHUB
  displayName: srcgraph-test
  deleteWhenDone: true
  maxRetries: 5
  retryTimeoutSeconds: 5
  config:
    gitHub:
      url: https://github.com
      orgs: []
      repos:
        - sourcegraph-testing/zap

searchQuery:
  - repo:^github.com/sourcegraph-testing/zap$ test
  - repo:^github.com/sourcegraph-testing/[email protected] test

insight:
  title: "Javascript to Typescript migration"
  dataSeries:
    [ {
      "query": "lang:javascript",
      "label": "javascript",
      "repositoryScope": [
        "github.com/sourcegraph/sourcegraph"
      ],
      "lineColor": "#6495ED",
      "timeScopeUnit": "MONTH",
      "timeScopeValue": 1
    },
      {
        "query": "lang:typescript",
        "label": "typescript",
        "lineColor": "#DE3163",
        "repositoryScope": [
          "github.com/sourcegraph/sourcegraph"
        ],
        "timeScopeUnit": "MONTH",
        "timeScopeValue": 1
      }
    ]
  deleteWhenDone: true

Test plan

Locally tested changes using latest Sourcegraph Helm install. Tested default configuration if no YAML is provided, YAML configuration provided, and JSON configuration provided.

ran go test ./... - all tests passing
ran staticcheck ./... - no issues found in new code

* Insights were automatically deleted, add option to keep insight created during validation
@jdpleiness jdpleiness self-assigned this Jan 23, 2023
Copy link
Contributor

@mucles mucles left a comment

Choose a reason for hiding this comment

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

Nice, this looks good to me 👍

@jdpleiness jdpleiness merged commit 79c8fa1 into main Jan 24, 2023
@jdpleiness jdpleiness deleted the jdp/validate-keep-insight branch January 24, 2023 19:43
scjohns pushed a commit that referenced this pull request Apr 24, 2023
* Insights were automatically deleted, add option to keep insight created during validation
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.

3 participants