Skip to content

add custom unit conversion#3

Merged
seveibar merged 2 commits intotscircuit:mainfrom
abhijitxy:main
Aug 30, 2024
Merged

add custom unit conversion#3
seveibar merged 2 commits intotscircuit:mainfrom
abhijitxy:main

Conversation

@abhijitxy
Copy link
Copy Markdown
Contributor

#2

@abhijitxy abhijitxy requested a review from seveibar August 30, 2024 18:14
@seveibar
Copy link
Copy Markdown
Contributor

Can you use tscircuit/plop to add a biome.json and a workflow for checking formatting to this repo. I noticed the formatting was a bit different which will cause problems for other contributors and also makes the diff difficult to read

@seveibar
Copy link
Copy Markdown
Contributor

@abhijitxy oh also can you add a bun-test workflow, also from tscircuit/plop? This will ensure that the tests pass

Copy link
Copy Markdown
Contributor

@seveibar seveibar left a comment

Choose a reason for hiding this comment

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

the code itself is awesome, but requesting changes while we setup the repo

@abhijitxy abhijitxy requested a review from seveibar August 30, 2024 19:03
Copy link
Copy Markdown
Contributor

@seveibar seveibar left a comment

Choose a reason for hiding this comment

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

beautiful

@seveibar seveibar merged commit cfe2c89 into tscircuit:main Aug 30, 2024
@homie-gg homie-gg Bot mentioned this pull request Aug 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants