Skip to content

adding intentional fail of style check#11

Open
vsoch wants to merge 1 commit into
developfrom
test/spack-bot
Open

adding intentional fail of style check#11
vsoch wants to merge 1 commit into
developfrom
test/spack-bot

Conversation

@vsoch
Copy link
Copy Markdown
Owner

@vsoch vsoch commented Jul 10, 2021

Signed-off-by: vsoch vsoch@users.noreply.github.com

Signed-off-by: vsoch <vsoch@users.noreply.github.com>
@vsoch vsoch closed this Jul 10, 2021
@vsoch vsoch reopened this Jul 10, 2021
@vsoch
Copy link
Copy Markdown
Owner Author

vsoch commented Jul 10, 2021

@spackbot hello

@spack-bot-develop
Copy link
Copy Markdown

👋️

@spack-bot-develop
Copy link
Copy Markdown

It looks like you had an issue with style checks! To fix this, you can run:

$ spack style --fix

And then update the pull request here.

@vsoch
Copy link
Copy Markdown
Owner Author

vsoch commented Jul 10, 2021

@spackbot commands

@spack-bot-develop
Copy link
Copy Markdown

You can interact with me in many ways!

  • @spackbot hello: say hello and get a friendly response back!
  • @spackbot help or @spackbot commands: see this message

I'll also help to label your pull request and assign reviewers!
If you need help or see there might be an issue with me, open an issue here

@vsoch
Copy link
Copy Markdown
Owner Author

vsoch commented Jul 13, 2021

@spackbot hello!

@spack-bot-develop
Copy link
Copy Markdown

こんにちは!

@vsoch
Copy link
Copy Markdown
Owner Author

vsoch commented Jul 13, 2021

@spackbot commands

@spack-bot-develop
Copy link
Copy Markdown

You can interact with me in many ways!

  • @spackbot hello: say hello and get a friendly response back!
  • @spackbot help or @spackbot commands: see this message
  • @spackbot fix style: ask me to fix a failed style check

I'll also help to label your pull request and assign reviewers!
If you need help or see there might be an issue with me, open an issue here

@vsoch
Copy link
Copy Markdown
Owner Author

vsoch commented Jul 13, 2021

@spackbot fix style!

@spack-bot-develop
Copy link
Copy Markdown

I cannot do that for you yet, Vanessa.

@spack-bot-develop
Copy link
Copy Markdown

Let me see if I can fix that for you!

@vsoch
Copy link
Copy Markdown
Owner Author

vsoch commented Jul 13, 2021

@spackbot fix style!

Repository owner deleted a comment from spack-bot-develop Bot Jul 13, 2021
@spack-bot-develop
Copy link
Copy Markdown

Let me see if I can fix that for you! This might take a moment...

@vsoch
Copy link
Copy Markdown
Owner Author

vsoch commented Jul 13, 2021

@spackbot fix style

Repository owner deleted a comment from spack-bot-develop Bot Jul 13, 2021
Repository owner deleted a comment from spack-bot-develop Bot Jul 13, 2021
@spack-bot-develop

This comment has been minimized.

Repository owner deleted a comment from spack-bot-develop Bot Jul 13, 2021
@spack-bot-develop
Copy link
Copy Markdown

Let me see if I can fix that for you! This might take a moment...

1 similar comment
@spack-bot-develop
Copy link
Copy Markdown

Let me see if I can fix that for you! This might take a moment...

@vsoch
Copy link
Copy Markdown
Owner Author

vsoch commented Jul 13, 2021

@spackbot fix style

@spack-bot-develop
Copy link
Copy Markdown

Let me see if I can fix that for you! This might take a moment...

2 similar comments
@spack-bot-develop
Copy link
Copy Markdown

Let me see if I can fix that for you! This might take a moment...

@spack-bot-develop
Copy link
Copy Markdown

Let me see if I can fix that for you! This might take a moment...

@vsoch
Copy link
Copy Markdown
Owner Author

vsoch commented Jul 13, 2021

@spackbot fix style again

@spack-bot-develop
Copy link
Copy Markdown

Let me see if I can fix that for you! This might take a moment...

@spack-bot-develop
Copy link
Copy Markdown

spack-bot-develop Bot commented Jul 13, 2021

I was able to run spack style --fix for you!

    ==> Running style checks on spack:
  selected: isort, mypy, flake8
==> Modified files:
  lib/spack/spack/cmd/style.py
==> Running isort checks
  isort checks were clean
==> Running mypy checks
  Success: no issues found in 508 source files
  mypy checks were clean
==> Running flake8 checks
  lib/spack/spack/cmd/style.py:30: [W291] trailing whitespace
  flake8 found errors

Keep in mind that I cannot fix your flake8 errors, so if you have any you'll need to fix them and update the pull request."

I wasn't able to make any changes, but please see the message above for any remaining issues!

@vsoch
Copy link
Copy Markdown
Owner Author

vsoch commented Jul 13, 2021

okay @spackbot once more - @spackbot fix style!

@spack-bot-develop
Copy link
Copy Markdown

Let me see if I can fix that for you! This might take a moment...

@spack-bot-develop
Copy link
Copy Markdown

I was able to run spack style --fix for you!

==> Running style checks on spack:
  selected: isort, mypy, flake8
==> Modified files:
  lib/spack/spack/cmd/style.py
==> Running isort checks
  isort checks were clean
==> Running mypy checks
  Success: no issues found in 508 source files
  mypy checks were clean
==> Running flake8 checks
  lib/spack/spack/cmd/style.py:30: [W291] trailing whitespace
  flake8 found errors

Keep in mind that I cannot fix your flake8 errors, so if you have any you'll need to fix them and update the pull request.
I wasn't able to make any changes, but please see the message above for any remaining issues!

@vsoch
Copy link
Copy Markdown
Owner Author

vsoch commented Jul 13, 2021

@spackbot fix style

@spack-bot-develop
Copy link
Copy Markdown

Let me see if I can fix that for you! This might take a moment...

@spack-bot-develop
Copy link
Copy Markdown

I was able to run spack style --fix for you!

==> Running style checks on spack:
  selected: isort, mypy, flake8
==> Modified files:
  lib/spack/spack/cmd/style.py
==> Running isort checks
  isort checks were clean
==> Running mypy checks
  Success: no issues found in 508 source files
  mypy checks were clean
==> Running flake8 checks
  lib/spack/spack/cmd/style.py:30: [W291] trailing whitespace
  flake8 found errors

Keep in mind that I cannot fix your flake8 or mypy errors, so if you have any you'll need to fix them and update the pull request.
I wasn't able to make any changes, but please see the message above for any remaining issues!

@vsoch
Copy link
Copy Markdown
Owner Author

vsoch commented Jul 14, 2021

@spackbot helloooooo

@spack-bot-develop
Copy link
Copy Markdown

👋️

@vsoch
Copy link
Copy Markdown
Owner Author

vsoch commented Jul 14, 2021

@spackbot fix style!

@spack-bot-develop
Copy link
Copy Markdown

Let me see if I can fix that for you! This might take a moment...

1 similar comment
@spack-bot-develop
Copy link
Copy Markdown

Let me see if I can fix that for you! This might take a moment...

@vsoch
Copy link
Copy Markdown
Owner Author

vsoch commented Jul 14, 2021

@spackbot fix style

@spack-bot-develop
Copy link
Copy Markdown

Let me see if I can fix that for you! This might take a moment...

@spack-bot-develop
Copy link
Copy Markdown

I was able to run spack style --fix for you!

==> Running style checks on spack:
  selected: isort, mypy, flake8
==> Modified files:
==> Running isort checks
  isort checks were clean
==> Running mypy checks
  Success: no issues found in 508 source files
  mypy checks were clean
==> Running flake8 checks
  flake8 checks were clean
==> spack style checks were clean

Keep in mind that I cannot fix your flake8 or mypy errors, so if you have any you'll need to fix them and update the pull request.
I wasn't able to make any changes, but please see the message above for any remaining issues!

@vsoch
Copy link
Copy Markdown
Owner Author

vsoch commented Jul 14, 2021

@spackbot fix style

@spack-bot-develop
Copy link
Copy Markdown

Let me see if I can fix that for you! This might take a moment...

@spack-bot-develop
Copy link
Copy Markdown

Here is the output from spack style --fix, which I ran for you!

==> Running style checks on spack:
  selected: isort, mypy, flake8
==> Modified files:
  lib/spack/spack/cmd/style.py
==> Running isort checks
  isort checks were clean
==> Running mypy checks
  Success: no issues found in 508 source files
  mypy checks were clean
==> Running flake8 checks
  lib/spack/spack/cmd/style.py:30: [W291] trailing whitespace
  flake8 found errors

Keep in mind that I cannot fix your flake8 or mypy errors, so if you have any you'll need to fix them and update the pull request.
I wasn't able to make any changes, but please see the message above for any remaining issues!

@vsoch
Copy link
Copy Markdown
Owner Author

vsoch commented Jul 19, 2021

@spackbot fix style!

@spack-bot-develop
Copy link
Copy Markdown

Let me see if I can fix that for you! This might take a moment...

@spack-bot-develop
Copy link
Copy Markdown

I was able to run spack style --fix for you!

spack style --fix
==> Running style checks on spack:
  selected: isort, mypy, flake8
==> Modified files:
  lib/spack/spack/cmd/style.py
==> Running isort checks
  isort checks were clean
==> Running mypy checks
  Success: no issues found in 508 source files
  mypy checks were clean
==> Running flake8 checks
  lib/spack/spack/cmd/style.py:30: [W291] trailing whitespace
  flake8 found errors
Keep in mind that I cannot fix your flake8 or mypy errors, so if you have any you'll need to fix them and update the pull request. I wasn't able to make any changes, but please see the message above for any remaining issues!

@vsoch
Copy link
Copy Markdown
Owner Author

vsoch commented Jul 27, 2021

@spackbot hello!

1 similar comment
@vsoch
Copy link
Copy Markdown
Owner Author

vsoch commented Jul 27, 2021

@spackbot hello!

@vsoch
Copy link
Copy Markdown
Owner Author

vsoch commented Jul 27, 2021

@spackbot hello again

@spack-bot-develop
Copy link
Copy Markdown

Hola!

@vsoch
Copy link
Copy Markdown
Owner Author

vsoch commented Jul 27, 2021

@spackbot fix style

@spack-bot-develop
Copy link
Copy Markdown

Let me see if I can fix that for you! This might take a moment...

@spack-bot-develop
Copy link
Copy Markdown

I was able to run spack style --fix for you!

spack style --fix
==> Running style checks on spack
  selected: isort, mypy, flake8
==> Modified files
==> Running isort checks
  isort checks were clean
==> Running mypy checks
  Success: no issues found in 514 source files
  mypy checks were clean
==> Running flake8 checks
  flake8 checks were clean
==> spack style checks were clean
Keep in mind that I cannot fix your flake8 or mypy errors, so if you have any you'll need to fix them and update the pull request. I wasn't able to make any further changes, but please see the message above for remaining issues you can fix locally!

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.

1 participant