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

Attach #1497

Merged
merged 92 commits into from
Jun 3, 2020
Merged

Attach #1497

merged 92 commits into from
Jun 3, 2020

Conversation

asvetlov
Copy link
Contributor

@asvetlov asvetlov commented May 7, 2020

No description provided.

@lgtm-com
Copy link

lgtm-com bot commented May 12, 2020

This pull request introduces 1 alert when merging 26b577f into 1057b68 - view on LGTM.com

new alerts:

  • 1 for Module is imported with 'import' and 'import from'

@lgtm-com
Copy link

lgtm-com bot commented May 13, 2020

This pull request introduces 1 alert when merging 79a84c3 into 1057b68 - view on LGTM.com

new alerts:

  • 1 for Module is imported with 'import' and 'import from'

@lgtm-com
Copy link

lgtm-com bot commented May 19, 2020

This pull request introduces 1 alert when merging bfc4332 into 186b424 - view on LGTM.com

new alerts:

  • 1 for Module is imported with 'import' and 'import from'

@codecov
Copy link

codecov bot commented May 20, 2020

Codecov Report

Merging #1497 into master will increase coverage by 0.06%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1497      +/-   ##
==========================================
+ Coverage   79.25%   79.31%   +0.06%     
==========================================
  Files          53       53              
  Lines        7799     7799              
  Branches     1293     1293              
==========================================
+ Hits         6181     6186       +5     
+ Misses       1413     1409       -4     
+ Partials      205      204       -1     
Impacted Files Coverage Δ
neuromation/api/jobs.py 87.11% <0.00%> (+0.25%) ⬆️
neuromation/cli/asyncio_utils.py 77.37% <0.00%> (+2.91%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e879f01...dd6dd03. Read the comment docs.

@lgtm-com
Copy link

lgtm-com bot commented May 20, 2020

This pull request introduces 1 alert when merging c999308 into 186b424 - view on LGTM.com

new alerts:

  • 1 for Module is imported with 'import' and 'import from'

@lgtm-com
Copy link

lgtm-com bot commented May 20, 2020

This pull request introduces 1 alert when merging 9b02f95 into e836239 - view on LGTM.com

new alerts:

  • 1 for Module is imported with 'import' and 'import from'

@lgtm-com
Copy link

lgtm-com bot commented May 20, 2020

This pull request introduces 2 alerts when merging 8fc60f5 into e836239 - view on LGTM.com

new alerts:

  • 1 for Unused import
  • 1 for Module is imported with 'import' and 'import from'

@lgtm-com
Copy link

lgtm-com bot commented May 20, 2020

This pull request introduces 1 alert when merging 99220c5 into e836239 - view on LGTM.com

new alerts:

  • 1 for Module is imported with 'import' and 'import from'

@lgtm-com
Copy link

lgtm-com bot commented May 20, 2020

This pull request introduces 1 alert when merging b2cfa14 into e836239 - view on LGTM.com

new alerts:

  • 1 for Module is imported with 'import' and 'import from'

@lgtm-com
Copy link

lgtm-com bot commented May 21, 2020

This pull request introduces 1 alert when merging ee73987 into e836239 - view on LGTM.com

new alerts:

  • 1 for Module is imported with 'import' and 'import from'

@lgtm-com
Copy link

lgtm-com bot commented Jun 2, 2020

This pull request introduces 2 alerts and fixes 2 when merging b62786e into 4920565 - view on LGTM.com

new alerts:

  • 2 for Module is imported with 'import' and 'import from'

fixed alerts:

  • 1 for Unused local variable
  • 1 for Module is imported with 'import' and 'import from'

Copy link
Contributor

@serhiy-storchaka serhiy-storchaka left a comment

Choose a reason for hiding this comment

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

👍

@lgtm-com
Copy link

lgtm-com bot commented Jun 2, 2020

This pull request introduces 2 alerts and fixes 2 when merging 4456d88 into 4920565 - view on LGTM.com

new alerts:

  • 2 for Module is imported with 'import' and 'import from'

fixed alerts:

  • 1 for Unused local variable
  • 1 for Module is imported with 'import' and 'import from'

@lgtm-com
Copy link

lgtm-com bot commented Jun 2, 2020

This pull request introduces 2 alerts and fixes 2 when merging 2bc9289 into 4920565 - view on LGTM.com

new alerts:

  • 2 for Module is imported with 'import' and 'import from'

fixed alerts:

  • 1 for Unused local variable
  • 1 for Module is imported with 'import' and 'import from'

@lgtm-com
Copy link

lgtm-com bot commented Jun 3, 2020

This pull request introduces 2 alerts and fixes 2 when merging 97458ce into 4920565 - view on LGTM.com

new alerts:

  • 2 for Module is imported with 'import' and 'import from'

fixed alerts:

  • 1 for Unused local variable
  • 1 for Module is imported with 'import' and 'import from'

@lgtm-com
Copy link

lgtm-com bot commented Jun 3, 2020

This pull request introduces 2 alerts and fixes 2 when merging d980da8 into e879f01 - view on LGTM.com

new alerts:

  • 2 for Module is imported with 'import' and 'import from'

fixed alerts:

  • 1 for Unused local variable
  • 1 for Module is imported with 'import' and 'import from'

@lgtm-com
Copy link

lgtm-com bot commented Jun 3, 2020

This pull request introduces 2 alerts and fixes 2 when merging 8bbb771 into e879f01 - view on LGTM.com

new alerts:

  • 2 for Module is imported with 'import' and 'import from'

fixed alerts:

  • 1 for Unused local variable
  • 1 for Module is imported with 'import' and 'import from'

@lgtm-com
Copy link

lgtm-com bot commented Jun 3, 2020

This pull request introduces 2 alerts and fixes 2 when merging 009918d into e879f01 - view on LGTM.com

new alerts:

  • 2 for Module is imported with 'import' and 'import from'

fixed alerts:

  • 1 for Unused local variable
  • 1 for Module is imported with 'import' and 'import from'

@lgtm-com
Copy link

lgtm-com bot commented Jun 3, 2020

This pull request introduces 2 alerts and fixes 2 when merging 119e8e1 into e879f01 - view on LGTM.com

new alerts:

  • 2 for Module is imported with 'import' and 'import from'

fixed alerts:

  • 1 for Unused local variable
  • 1 for Module is imported with 'import' and 'import from'

@lgtm-com
Copy link

lgtm-com bot commented Jun 3, 2020

This pull request introduces 2 alerts and fixes 2 when merging ab7fe8b into e879f01 - view on LGTM.com

new alerts:

  • 2 for Module is imported with 'import' and 'import from'

fixed alerts:

  • 1 for Unused local variable
  • 1 for Module is imported with 'import' and 'import from'

@lgtm-com
Copy link

lgtm-com bot commented Jun 3, 2020

This pull request introduces 2 alerts and fixes 2 when merging dd6dd03 into e879f01 - view on LGTM.com

new alerts:

  • 2 for Module is imported with 'import' and 'import from'

fixed alerts:

  • 1 for Unused local variable
  • 1 for Module is imported with 'import' and 'import from'

@asvetlov asvetlov merged commit dfe74a9 into master Jun 3, 2020
@asvetlov asvetlov deleted the attach branch June 3, 2020 14:17
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