Skip to content

refactor: Move constants to the constants.ts file#8

Merged
ardalanamini merged 1 commit intomainfrom
refactor/move-constants
Jul 24, 2022
Merged

refactor: Move constants to the constants.ts file#8
ardalanamini merged 1 commit intomainfrom
refactor/move-constants

Conversation

@ardalanamini
Copy link
Copy Markdown
Member

📑 Description

  • Move constants to the constants.ts file

✅ Checks

  • My code requires changes to the documentation
  • I have updated the documentation as required

@ardalanamini ardalanamini added enhancement New feature or request typescript Pull requests that update Typescript code labels Jul 24, 2022
@ardalanamini ardalanamini added this to the v1.0.0 milestone Jul 24, 2022
@ardalanamini ardalanamini self-assigned this Jul 24, 2022
@codecov
Copy link
Copy Markdown

codecov bot commented Jul 24, 2022

Codecov Report

Merging #8 (bb3cc4f) into main (cfeea37) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main        #8   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            4         5    +1     
  Lines           69        70    +1     
  Branches         8         8           
=========================================
+ Hits            69        70    +1     
Impacted Files Coverage Δ
src/MiniCSS.ts 100.00% <ø> (ø)
src/index.ts 100.00% <ø> (ø)
src/constants.ts 100.00% <100.00%> (ø)
src/utils.ts 100.00% <100.00%> (ø)

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@ardalanamini ardalanamini merged commit 4765517 into main Jul 24, 2022
@ardalanamini ardalanamini deleted the refactor/move-constants branch July 24, 2022 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request typescript Pull requests that update Typescript code

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant