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.

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

Validate: add deleteWhenDone option to insights #927

Merged
merged 1 commit into from
Jan 24, 2023

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/zap$@v1.14.1 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
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.

2 participants