{tools}[GCCcore/13.3.0] ruff v0.7.0#23466
Conversation
…7.0_fix-lsp-types.patch
Flamefire
left a comment
There was a problem hiding this comment.
With the update of easybuilders/easybuild-easyblocks#3665 this works for me
| name = "lsp-types" | ||
| version = "0.95.1" | ||
| -source = "git+https://github.com/astral-sh/lsp-types.git?rev=3512a9f#3512a9f33eadc5402cfab1b8f7340824c8ca1439" | ||
| +source = "git+https://github.com/astral-sh/lsp-types.git?rev=3512a9f33eadc5402cfab1b8f7340824c8ca1439#3512a9f33eadc5402cfab1b8f7340824c8ca1439" |
There was a problem hiding this comment.
This patch shouldn't be required. The easyconfig can use ('lsp-types', '0.95.1', 'https://github.com/astral-sh/lsp-types.git', '3512a9f'), instead
We should update the cargo.py gather logic to avoid
ValueError: Found different revision in query of URL https://github.com/astral-sh/lsp-types.git: 3512a9f (expected: 3512a9f33eadc5402cfab1b8f7340824c8ca1439)
There was a problem hiding this comment.
Yes, that makes sense.. I wonder if it is the same issue as here:
#21392 (comment)
|
Test report by @Flamefire |
|
Test report by @pavelToman |
|
Test report by @pavelToman |
…it's no longer needed due to improved Cargo easyblock
|
Test report by @boegel |
|
@boegelbot please test @ jsc-zen3 |
|
@boegel: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3431066204 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
Going in, thanks @WilleBell! |
(created using
eb --new-pr)Depends on: