Skip to content

Preserve dependencies order#166

Merged
rtimush merged 1 commit intomasterfrom
push-lzosrkxmvksw
Oct 21, 2025
Merged

Preserve dependencies order#166
rtimush merged 1 commit intomasterfrom
push-lzosrkxmvksw

Conversation

@rtimush
Copy link
Collaborator

@rtimush rtimush commented Oct 21, 2025

When the same dependency exists multiple times in the dependency tree with slightly different attributes, we prefer the one closer to the root, but if they appear on the same level, the order matters (the first one wins).

The PR makes protofetch process dependencies in the order they are defined in protofetch.toml, which should make resolution reproducible.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@rtimush rtimush merged commit 16e1637 into master Oct 21, 2025
16 checks passed
@rtimush rtimush deleted the push-lzosrkxmvksw branch October 21, 2025 07:23
@rtimush rtimush mentioned this pull request Oct 21, 2025
rtimush added a commit that referenced this pull request Oct 21, 2025
## 🤖 New release

* `protofetch`: 0.1.11 -> 0.1.12

<details><summary><i><b>Changelog</b></i></summary><p>

<blockquote>

##
[0.1.12](v0.1.11...v0.1.12)
- 2025-10-21

### Other

- Preserve dependencies order
([#166](#166))
- Delete apple_sdk.frameworks.Security
([#165](#165))
- Add to the readme an example of subtree checkout
([#164](#164))
- Include content_roots in README
([#163](#163))
- Upgrade dependencies and tooling
([#160](#160))
- fix typo (#159)
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/release-plz/release-plz/).
Sign up for free to 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