-
Notifications
You must be signed in to change notification settings - Fork 15
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
chore(deps): fix dep versioning, conflicts, type #16
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM only a question if we should use undefined instead of null
source: "design-guidelines", | ||
tabName: null, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we use undefined instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, I think that tabName value is just coming from the docs framework, and it's coming back as null because we don't define a specific one. It's just showing up as a change here because I bumped the docs framework version.
But yeah that seems like it might be a change we want to make in the docs framework.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah that frontmatter value is coming from this PR in org https://github.com/patternfly/patternfly-org/pull/3394/files
🎉 This PR is included in version 1.0.0-prerelease.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
🎉 This PR is included in version 4.87.101 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Addresses dependency concerns in #14