Skip to content
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

feat/HEYUI-80 Accordion component #51

Merged
merged 56 commits into from
Sep 1, 2022
Merged

Conversation

derek-heycar
Copy link
Contributor

Pull Request

Description

Describe your changes (or add ticket description) and add relevant screenshots

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Documentation change

How do I test this

  • Add steps to test
  • in bullet point format
  • preferably you can add link to the storybook build in the PR

Checklist

Did you remember to take care of the following?

  • npm i – for new NPM dependencies.
  • npm run lint - to check for linting issues
  • npm run test - to run unit tests
  • npm run test:screenshots - to run snapshot tests

New Feature / Bug Fix

  • Run unit tests to ensure all existing tests are still passing.
  • Add new passing unit tests to cover the code introduced by your pr.

Thanks for contributing!

derek-heycar and others added 19 commits August 2, 2022 15:14
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
added basic implementation of chip component with choice and filter options
added icon.module.css to put icon styles in a seperate file
updated component.stories with text and removed redundent prop
added snapshot and unit tests for chip component
@auto-assign auto-assign bot requested a review from hcafaq August 16, 2022 13:57
@github-actions
Copy link

Compiled a new version demo.

@coveralls
Copy link

coveralls commented Aug 16, 2022

Pull Request Test Coverage Report for Build 2971426610

  • 41 of 43 (95.35%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.8%) to 98.507%

Changes Missing Coverage Covered Lines Changed/Added Lines %
packages/accordion/src/Accordion.tsx 21 23 91.3%
Totals Coverage Status
Change from base Build 2949609739: 0.8%
Covered Lines: 698
Relevant Lines: 700

💛 - Coveralls

@github-actions
Copy link

github-actions bot commented Aug 16, 2022

Coverage report

❌ An unexpected error occurred. For more details, check console

Error: The process '/usr/bin/git' failed with exit code 1
St.
Category Percentage Covered / Total
🟢 Statements 99.33% 746/751
🟢 Branches 90.38% 94/104
🟢 Functions 97.3% 36/37
🟢 Lines 99.71% 698/700

Test suite run success

155 tests passing in 24 suites.

Report generated by 🧪jest coverage report action from 9e49033

@github-actions
Copy link

Compiled a new version demo.

Replace the font sets regular, bold, and extra bold. Added extra bold font for the individual import
/600

BREAKING CHANGE: Can be changed the width of the font for some of the packages which use the font
setting
@github-actions
Copy link

Compiled a new version demo.

hcafaq and others added 3 commits August 28, 2022 21:32
…hey-ui into feature/HEYUI-130-implementation-of-the-chips-component
resolved css lint issues in icon.module.css in chip component
@github-actions
Copy link

Compiled a new version demo.

@github-actions
Copy link

Compiled a new version demo.

@github-actions
Copy link

github-actions bot commented Sep 1, 2022

Compiled a new version demo.

@github-actions
Copy link

github-actions bot commented Sep 1, 2022

Compiled a new version demo.

 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
 - @heycar-uikit/[email protected]
@github-actions
Copy link

github-actions bot commented Sep 1, 2022

Compiled a new version demo.

@github-actions
Copy link

github-actions bot commented Sep 1, 2022

Compiled a new version demo.

@derek-heycar derek-heycar merged commit b8d3f3f into main Sep 1, 2022
@derek-heycar derek-heycar deleted the feat/HEYUI-80-accordion branch September 1, 2022 12:07
derek-heycar pushed a commit that referenced this pull request Sep 1, 2022
# [1.14.0](v1.13.0...v1.14.0) (2022-09-01)

### Features

* **collapse:** the first version of the collapse component ([#51](#51)) ([b8d3f3f](b8d3f3f))
@derek-heycar
Copy link
Contributor Author

🎉 This PR is included in version 1.14.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants