Skip to content

chore: update o-labels#849

Merged
metart43 merged 1 commit intomainfrom
am/update-o-labels
Feb 17, 2026
Merged

chore: update o-labels#849
metart43 merged 1 commit intomainfrom
am/update-o-labels

Conversation

@metart43
Copy link
Copy Markdown
Member

What's changed

  • Need to bump o-labels dependency to unblock Storytelling. They cannot merge PRs due to dependency conflict.

Comment thread components/x-gift-article/package.json Outdated
"@financial-times/o-banner": "^5.0.0",
"@financial-times/o-forms": "^10.0.1",
"@financial-times/o-labels": "^7.0.0",
"@financial-times/o-labels": "^8.0.0",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

is this one where we want to say >=7?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I am a bit hesitant to go >=7. What if v9 breaks gift article somehow?

I could do "@financial-times/o-labels": "^7.0.0 || ^8.0.0" instead

@metart43 metart43 merged commit 5027e03 into main Feb 17, 2026
13 checks passed
@metart43 metart43 deleted the am/update-o-labels branch February 17, 2026 10:48
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