Skip to content

Conversation

jamesbraza
Copy link
Collaborator

@jamesbraza jamesbraza commented Aug 25, 2025

#1075 was automerged before I could pull in the correct pinning

@jamesbraza jamesbraza self-assigned this Aug 25, 2025
@jamesbraza jamesbraza added the bug Something isn't working label Aug 25, 2025
@Copilot Copilot AI review requested due to automatic review settings August 25, 2025 22:29
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Aug 25, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes an issue with the LDP dependency version constraint by adding an upper bound to prevent potential breaking changes from version 1.0 and above.

Key Changes

  • Added upper bound constraint to LDP dependency to avoid future compatibility issues

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Aug 25, 2025
Copy link
Collaborator

@maykcaldas maykcaldas left a comment

Choose a reason for hiding this comment

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

🚀

@jamesbraza jamesbraza merged commit 4457624 into main Aug 25, 2025
5 checks passed
@jamesbraza jamesbraza deleted the fixing-ldp-pinning branch August 25, 2025 22:37
Copy link

dosubot bot commented Aug 25, 2025

Documentation updates
Checked 1 published document(s). No updates required.

How did I do? Any feedback?  Join Discord

jamesbraza added a commit that referenced this pull request Aug 25, 2025
@jamesbraza
Copy link
Collaborator Author

Crap I missed fhlmi, will just get that later

jamesbraza added a commit that referenced this pull request Aug 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants