Skip to content
This repository was archived by the owner on Jun 17, 2022. It is now read-only.

Conversation

@Hinton
Copy link
Member

@Hinton Hinton commented Jan 21, 2022

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

Adds the scaffolding for the Component Library. It's packaged as an Angular cli application to provide support for storybook. In the long term we should look into how we can run storybook outside jslib since it's currently limited to components within jslib, which is a bit limited.

Tailwind is configured to use the prefix tw- to avoid clashing with bootstrap.

This project is setup slightly different to most of our projects in that it has index.ts which exposes a public api. This is intentional as a way to define a public api.

Resolves:

  • CL-8
  • CL-9
  • CL-12

Before you submit

  • I have checked for linting errors (npm run lint) (required)
  • I have added unit tests where it makes sense to do so (encouraged but not required)
  • This change requires a documentation update (notify the documentation team)
  • This change has particular deployment requirements (notify the DevOps team)

@Hinton Hinton marked this pull request as ready for review February 4, 2022 14:21
@Hinton Hinton changed the title Component Library Scaffolding Component Library scaffolding Feb 4, 2022
danielleflinn
danielleflinn previously approved these changes Mar 1, 2022
Copy link

@danielleflinn danielleflinn left a comment

Choose a reason for hiding this comment

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

Visual reviewed via Chromatic and Storybook; looks good to me.

vincentsalucci
vincentsalucci previously approved these changes Mar 5, 2022
@Hinton Hinton requested a review from vincentsalucci March 7, 2022 13:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants