Skip to content

Add: py-sphinx-design#32482

Merged
ax3l merged 2 commits intospack:developfrom
ax3l:add-sphinx-design
Sep 1, 2022
Merged

Add: py-sphinx-design#32482
ax3l merged 2 commits intospack:developfrom
ax3l:add-sphinx-design

Conversation

@ax3l
Copy link
Copy Markdown
Member

@ax3l ax3l commented Sep 1, 2022

Needed for #32480

@ax3l ax3l requested a review from adamjstewart September 1, 2022 18:40
@ax3l ax3l added the documentation Improvements or additions to documentation label Sep 1, 2022
@adamjstewart adamjstewart self-assigned this Sep 1, 2022
@ax3l ax3l marked this pull request as ready for review September 1, 2022 18:41
@ax3l ax3l force-pushed the add-sphinx-design branch from 2afcfb9 to f5d5506 Compare September 1, 2022 18:48
@ax3l ax3l requested a review from adamjstewart September 1, 2022 18:49
@adamjstewart

This comment was marked as outdated.

@spackbot-app

This comment was marked as outdated.

@ax3l ax3l force-pushed the add-sphinx-design branch from f5d5506 to a200f9c Compare September 1, 2022 18:50
@ax3l
Copy link
Copy Markdown
Member Author

ax3l commented Sep 1, 2022

@spackbot fix style

@spackbot-app
Copy link
Copy Markdown

spackbot-app bot commented Sep 1, 2022

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

spack style --fix
==> Running style checks on spack
  selected: isort, mypy, black, flake8
==> Modified files
  var/spack/repos/builtin/packages/py-sphinx-design/package.py
==> Running isort checks
  isort checks were clean
==> Running mypy checks
Success: no issues found in 558 source files
  mypy checks were clean
==> Running black checks
reformatted var/spack/repos/builtin/packages/py-sphinx-design/package.py
All done! ✨ 🍰 ✨
1 file reformatted.
  black 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. If I was able to push to your branch, if you make further changes you will need to pull from your updated branch before pushing again.

I've updated the branch with isort fixes.

@spackbot-app
Copy link
Copy Markdown

spackbot-app bot commented Sep 1, 2022

Let me see if I can fix that for you!

@ax3l ax3l enabled auto-merge (squash) September 1, 2022 18:53
@adamjstewart
Copy link
Copy Markdown
Member

I've updated the branch with isort fixes.

@scottwittenburg this should say "black fixes", not "isort fixes".

@ax3l ax3l merged commit ce1500f into spack:develop Sep 1, 2022
@ax3l ax3l deleted the add-sphinx-design branch September 1, 2022 20:02
ma595 pushed a commit to ma595/spack that referenced this pull request Sep 13, 2022
* Add: py-sphinx-design

Needed for spack#32480
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants