Skip to content

refactor: Mark internal and python extension as reproducible - #1892

Merged
aignas merged 5 commits into
bazel-contrib:mainfrom
fmeum:reproducible
May 18, 2024
Merged

refactor: Mark internal and python extension as reproducible#1892
aignas merged 5 commits into
bazel-contrib:mainfrom
fmeum:reproducible

Conversation

@fmeum

@fmeumfmeum commented May 12, 2024

Copy link
Copy Markdown
Member

This avoids unnecessary lockfile entries for users of rules_python.

This avoids an unnecessary lockfile entry for users of rules_python.
@fmeum
fmeum marked this pull request as ready for review May 12, 2024 21:37
@fmeum
fmeum requested a review from rickeylev as a code ownerMay 12, 2024 21:37
@fmeum

Copy link
Copy Markdown
MemberAuthor

@aignas Could you review this?

@aignasaignas left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM in general, would love a line in CHANGELOG about the bumped dependency.

It does mean that the rules_python_gazelle_plugin will always have a warning because it depends on 1.3, so maybe we should bump the gazelle/MODULE.bazel as well?

Comment threadMODULE.bazel
@fmeum
fmeum requested a review from f0rmiga as a code ownerMay 16, 2024 08:45
@fmeum
fmeum requested a review from aignasMay 16, 2024 08:46
@fmeumfmeum changed the title refactor: Mark internal deps extension as reproduciblerefactor: Mark some extensions as reproducibleMay 16, 2024
@fmeumfmeum changed the title refactor: Mark some extensions as reproduciblerefactor: Mark internal and python extension as reproducibleMay 16, 2024
@fmeum

fmeum commented May 16, 2024

Copy link
Copy Markdown
MemberAuthor

I also marked the python extension as reproducible. pip is of course a different story, so I haven't touched it.

@aignas
aignas added this pull request to the merge queueMay 18, 2024
Merged via the queue into bazel-contrib:main with commit 45363a1May 18, 2024
@fmeum
fmeum deleted the reproducible branch May 19, 2024 08:24
Sign up for freeto 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.

2 participants

@fmeum@aignas