-
Notifications
You must be signed in to change notification settings - Fork 0
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: DLT-1941 add dialtone combinator package #8
base: staging
Are you sure you want to change the base?
Conversation
Signed-off-by: Julio Ortega <[email protected]>
…ialpad#500) Signed-off-by: Julio Ortega <[email protected]>
Signed-off-by: Julio Ortega <[email protected]>
…ialpad#500) Signed-off-by: Julio Ortega <[email protected]>
Signed-off-by: Julio Ortega <[email protected]>
…add_dt-combinator_package # Conflicts: # pnpm-lock.yaml
Signed-off-by: Julio Ortega <[email protected]>
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.
PR Summary
This pull request introduces the Dialtone Combinator package to the monorepo, focusing on integration and preparation for future development.
- Added
packages/combinator
to.markdownlintignore
and.stylelintignore
to exclude it from linting - Simplified
ComponentCombinator.vue
, now usingdtc-combinator
for rendering - Updated
apps/dialtone-documentation/docs/.vuepress/config.js
to accommodate Combinator integration - Modified
apps/dialtone-documentation/docs/.vuepress/theme/client.js
to include Combinator CSS and register components - Added '@dialpad/dialtone-combinator' as a workspace dependency in
apps/dialtone-documentation/package.json
12 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings
'pnpm exec stylelint --fix --allow-empty-input', | ||
], | ||
'*.{md, mdx}': [ | ||
'pnpm exec markdownlint' | ||
] | ||
'pnpm exec markdownlint', | ||
], |
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.
style: Consider using a consistent style for trailing commas in the configuration object
[NO-MERGE] Add Dialtone Combinator package to monorepo
Obligatory GIF (super important!)
🛠️ Type Of Change
These types will not increment the version number, but will still deploy to documentation site on release:
📖 Jira Ticket
https://dialpad.atlassian.net/browse/DLT-1941
📖 Description
.markdownlintignore
and.stylelintignore
💡 Context
Starting the investigation to make the dialtone-combinator work on our documentation site.
📝 Checklist
For all PRs:
For all Vue changes:
./scripts/dialtone-vue-sync.sh
script. Read docs here: Dialtone Vue Sync ScriptFor all CSS changes:
If new component:
packages/dialtone-vue2
orpackages/dialtone-vue3
).packages/dialtone-css
package.apps/dialtone-documentation
.common/components_list.js
🔮 Next Steps