Skip to content

Commit 7edac99

Browse files
authored
Ignore deps starting with uv when finding uv version (#492)
Fixes: #489
1 parent 05273c1 commit 7edac99

4 files changed

Lines changed: 15 additions & 4 deletions

File tree

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1 +1,2 @@
1+
uvicorn==0.35.0
12
uv==0.6.17
Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
import { getUvVersionFromFile } from "../../src/version/resolve";
2+
3+
import { expect, test } from "@jest/globals";
4+
5+
test("ignores dependencies starting with uv", async () => {
6+
const parsedVersion = getUvVersionFromFile(
7+
"__tests__/fixtures/uv-in-requirements-txt-project/requirements.txt",
8+
);
9+
expect(parsedVersion).toBe("0.6.17");
10+
});

dist/setup/index.js

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/version/requirements-file.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@ function getUvVersionFromAllDependencies(
1515
allDependencies: string[],
1616
): string | undefined {
1717
return allDependencies
18-
.find((dep: string) => dep.startsWith("uv"))
19-
?.match(/^uv([^A-Z0-9._-]+.*)$/)?.[1]
18+
.find((dep: string) => dep.match(/^uv[=<>~!]/))
19+
?.match(/^uv([=<>~!]+.*)$/)?.[1]
2020
.trim();
2121
}
2222

0 commit comments

Comments
 (0)