Skip to content

Conversation

hawkw
Copy link
Member

@hawkw hawkw commented Apr 7, 2025

PR #7907 broke the Nix flake, as it removed the
tools/dendrite_openapi_version file and replaced it with tools/dendrite_version. The flake was previously using tools/dendrite_openapi_version to determine both the Dendrite OpenAPI commit and the Dendrite stub commit to download. This commit changes it to use the new dendrite_version file to determine the commit for downloading the Dendrite stub, and removed the code for downloading the Dendrite OpenAPI document.

PR #7907 broke the Nix flake, as it removed the
`tools/dendrite_openapi_version` file and replaced it with
`tools/dendrite_version`. The flake was previously using
`tools/dendrite_openapi_version` to determine both the Dendrite OpenAPI
commit and the Dendrite stub commit to download. This commit changes it
to use the new `dendrite_version` file to determine the commit for
downloading the Dendrite stub, and removed the code for downloading the
Dendrite OpenAPI document.
@hawkw hawkw requested review from iliana and sunshowers April 7, 2025 20:31
Copy link
Contributor

@sunshowers sunshowers left a comment

Choose a reason for hiding this comment

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

Always love a prime

@hawkw
Copy link
Member Author

hawkw commented Apr 7, 2025

Always love a prime

@sunshowers it felt contextually appropriate :)

@hawkw hawkw enabled auto-merge (squash) April 7, 2025 20:43
@hawkw hawkw merged commit d9f9337 into main Apr 8, 2025
16 checks passed
@hawkw hawkw deleted the eliza/fix-flake branch April 8, 2025 04:00
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.

3 participants