Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Sign up
Appearance settings

bzlmod: allow custom node_repositories definitions #3802

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

Draft
pjjw wants to merge 1 commit into bazel-contrib:main
base: main
Choose a base branch
Loading
from pjjw:pjjw/extension-custom-versions

Conversation

@pjjw
Copy link

@pjjw pjjw commented Nov 10, 2024

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature (please, look at the "Scope of the project" section in the README.md file)
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

Silic0nS0ldier reacted with thumbs up emoji
@pjjw pjjw force-pushed the pjjw/extension-custom-versions branch from 4ee8284 to 36434c4 Compare November 10, 2024 20:38
Copy link
Contributor

guw commented May 20, 2025

@pjjw Is this ready for review or do you intend to add more changed to the PR?

Copy link

Silic0nS0ldier commented Jun 27, 2025 •
edited
Loading

#3843 merged, this can be closed.

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

Reviewers

@mattem mattem Awaiting requested review from mattem mattem will be requested when the pull request is marked ready for review mattem is a code owner

@gregmagolan gregmagolan Awaiting requested review from gregmagolan gregmagolan will be requested when the pull request is marked ready for review gregmagolan is a code owner

@alexeagle alexeagle Awaiting requested review from alexeagle alexeagle will be requested when the pull request is marked ready for review alexeagle is a code owner

@jbedard jbedard Awaiting requested review from jbedard jbedard will be requested when the pull request is marked ready for review jbedard is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /