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

Remove lgtm tooling #4300

Merged
merged 1 commit into from
Jul 14, 2021
Merged

Remove lgtm tooling #4300

merged 1 commit into from
Jul 14, 2021

Conversation

jngrad
Copy link
Member

@jngrad jngrad commented Jul 14, 2021

The lgtm tool is currently failing at the Extraction stage of the Python analysis. It happens frequently and at random. The logfile is simply cut after 3 minutes and the lgtm runner doesn't respond for 4 hours. The logfile doesn't provide enough information to help us diagnose the root cause of the runner failure.

The link to the support page in the GitHub Marketplace page returns a 404 error and was crawled for the last time in 2019 by the Wayback Machine. Attempts at fixing the issue were unfruitful (8c68ca2, e1b79f1). This PR removes lgtm integration.

@jngrad jngrad added the DevOps label Jul 14, 2021
@jngrad jngrad added the automerge Merge with kodiak label Jul 14, 2021
@kodiakhq kodiakhq bot merged commit fdea176 into espressomd:python Jul 14, 2021
@jngrad jngrad deleted the remove-lgtm branch July 14, 2021 11:18
@jngrad jngrad added this to the Espresso 4.2 milestone Jun 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge with kodiak DevOps
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants