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

dla-future-fortran: new package #44314

Merged
merged 7 commits into from
May 24, 2024
Merged

Conversation

RMeli
Copy link
Member

@RMeli RMeli commented May 22, 2024

@rasolca
Copy link
Contributor

rasolca commented May 22, 2024

@spackbot fix style

Copy link

spackbot-app bot commented May 22, 2024

Let me see if I can fix that for you!

Copy link

spackbot-app bot commented May 22, 2024

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

spack style --fix
==> Running style checks on spack
  selected: isort, black, flake8, mypy
==> Modified files
  var/spack/repos/builtin/packages/dla-future-fortran/package.py
==> Running isort checks
Fixing /tmp/tmp4_o6hhwz/spack/var/spack/repos/builtin/packages/dla-future-fortran/package.py
  isort checks were clean
==> Running black checks
All done! ✨ 🍰 ✨
1 file left unchanged.
  black checks were clean
==> Running flake8 checks
  flake8 checks were clean
==> Running mypy checks
lib/spack/spack/version/version_types.py:145: error: Argument 2 to "StandardVersion" has incompatible type "*Tuple[Tuple[Any, ...], Tuple[Any, ...]]"; expected "Tuple[Tuple[Any, ...], Tuple[Any, ...]]"  [arg-type]
lib/spack/spack/version/version_types.py:452: error: Argument 2 to "StandardVersion" has incompatible type "*Tuple[Tuple[Any, ...], Tuple[Any, ...]]"; expected "Tuple[Tuple[Any, ...], Tuple[Any, ...]]"  [arg-type]
lib/spack/spack/version/version_types.py:481: error: Argument 2 to "StandardVersion" has incompatible type "*Tuple[Tuple[Any, ...], Tuple[Any, ...]]"; expected "Tuple[Tuple[Any, ...], Tuple[Any, ...]]"  [arg-type]
Found 3 errors in 1 file (checked 625 source files)
  mypy 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. 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 style fixes.

msimberg
msimberg previously approved these changes May 22, 2024
@albestro albestro disabled auto-merge May 23, 2024 08:44
@RMeli RMeli requested review from msimberg and haampie May 23, 2024 15:27
@RMeli RMeli requested a review from aurianer May 23, 2024 15:27
@alecbcs alecbcs changed the title DLA-Future-Fortran: new package dla-future-fortran: new package May 24, 2024
@albestro albestro merged commit 8aa490d into spack:develop May 24, 2024
14 checks passed
@RMeli RMeli deleted the dla-future-fortran branch May 24, 2024 07:53
teaguesterling pushed a commit to teaguesterling/spack that referenced this pull request Jun 15, 2024
* Spglib: add version 2.4.0

* dla-future-fortran: new package version 0.1.0

* [@spackbot] updating style on behalf of RMeli

* apply suggestion and add maintainer

---------

Co-authored-by: RMeli <RMeli@users.noreply.github.com>
hariharan-devarajan pushed a commit to hariharan-devarajan/spack that referenced this pull request Jul 10, 2024
* Spglib: add version 2.4.0

* dla-future-fortran: new package version 0.1.0

* [@spackbot] updating style on behalf of RMeli

* apply suggestion and add maintainer

---------

Co-authored-by: RMeli <RMeli@users.noreply.github.com>
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.

6 participants