-
Notifications
You must be signed in to change notification settings - Fork 11
feat: change configuration file format #811
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
Closed
sileht
wants to merge
1
commit into
devs/sileht/fetch-for-diff/Ie50484903e93782a7b3b329251b72ef75979b608
from
devs/sileht/fetch-for-diff/I2e4195e7fc8ce38325b618573456d26b05a57b56
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This was referenced Oct 10, 2025
Member
Author
Contributor
Merge ProtectionsYour pull request matches the following merge protections and will not be merged until they are valid. 🔴 ⛓️ Depends-On RequirementsThis rule is failing.Requirement based on the presence of
🔴 🤖 Continuous IntegrationThis rule is failing.
🔴 👀 Review RequirementsThis rule is failing.
🔴 🔎 ReviewsThis rule is failing.
🟢 Enforce conventional commitWonderful, this rule succeeded.Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
|
Contributor
🧪 CI InsightsHere's what we observed from your CI run for 29b95c6. 🟢 All jobs passed!But CI Insights is watching 👀 |
156470a to
c36f3d8
Compare
300a416 to
e2f1e39
Compare
c36f3d8 to
a6188ba
Compare
e2f1e39 to
0a833f0
Compare
a6188ba to
f7fe19d
Compare
0a833f0 to
a65c865
Compare
f7fe19d to
9bddfe7
Compare
a65c865 to
a9c2f35
Compare
9bddfe7 to
e6f458b
Compare
a9c2f35 to
d960a71
Compare
e6f458b to
961ab7a
Compare
d960a71 to
63f2bc4
Compare
This change changes the configuration format to:
```
scopes:
merge_queue_scope: merge-queue
mode: parallel
source:
files:
js:
include:
- "**/*.js"
python:
include:
- "**/*.py"
other:
include:
- "**/*"
exclude:
- "**/*.py"
- "**/*.js"
```
Sources will later have value like: `bazel`, `nx`, `turporepo`
We will have something like:
```
scopes:
merge_queue_scope: merge-queue
mode: parallel
source:
nx:
command: ["npx", "nx"]
```
To get the scopes, this will run: `npx nx show projects --affected --base={base} --head={head}"`
Change-Id: I2e4195e7fc8ce38325b618573456d26b05a57b56
961ab7a to
4589814
Compare
63f2bc4 to
29b95c6
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This change changes the configuration format to:
Sources will later have value like:
bazel,nx,turporepoWe will have something like:
To get the scopes, this will run:
npx nx show projects --affected --base={base} --head={head}"Depends-On: #808