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

Hotfixes #3846

Merged
merged 3 commits into from
Aug 3, 2020
Merged

Hotfixes #3846

merged 3 commits into from
Aug 3, 2020

Conversation

KaiSzuttor
Copy link
Member

No description provided.

@lgtm-com
Copy link

lgtm-com bot commented Aug 3, 2020

This pull request fixes 12 alerts when merging 56e77ed into c144f26 - view on LGTM.com

fixed alerts:

  • 6 for Suspicious unused loop iteration variable
  • 4 for Unused exception object
  • 1 for Nested loops with same variable
  • 1 for Nested loops with same variable reused after inner loop body

@KaiSzuttor KaiSzuttor added the automerge Merge with kodiak label Aug 3, 2020
@lgtm-com
Copy link

lgtm-com bot commented Aug 3, 2020

This pull request fixes 12 alerts when merging 217bfdc into c144f26 - view on LGTM.com

fixed alerts:

  • 6 for Suspicious unused loop iteration variable
  • 4 for Unused exception object
  • 1 for Nested loops with same variable
  • 1 for Nested loops with same variable reused after inner loop body

@jngrad
Copy link
Member

jngrad commented Aug 3, 2020

Is there a way to update the first lgtm-com post instead of posting a new message after each push? I couldn't find such as setting in their YAML docs not in their GitHub integration docs.

@KaiSzuttor
Copy link
Member Author

i just deactivated the comments, we don't need them

@kodiakhq kodiakhq bot merged commit ae1fcc2 into espressomd:python Aug 3, 2020
@jngrad jngrad added this to the Espresso 4.1.4 milestone Aug 18, 2020
jngrad pushed a commit to jngrad/espresso that referenced this pull request Aug 18, 2020
@jngrad jngrad added the BugFix label Aug 18, 2020
jngrad pushed a commit to jngrad/espresso that referenced this pull request Aug 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge with kodiak BugFix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants