Skip to content

Stricting camelCase#28

Merged
seveibar merged 5 commits intotscircuit:mainfrom
DhairyaMajmudar:casing
Oct 6, 2024
Merged

Stricting camelCase#28
seveibar merged 5 commits intotscircuit:mainfrom
DhairyaMajmudar:casing

Conversation

@DhairyaMajmudar
Copy link
Copy Markdown
Contributor

-> Adding rules for camelCase in biome.json file
-> related to #27

cc: @seveibar

@vercel
Copy link
Copy Markdown

vercel Bot commented Oct 6, 2024

@DhairyaMajmudar is attempting to deploy a commit to the tscircuit Team on Vercel.

A member of the Team first needs to authorize it.

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.

check out the link in the original issue, you adjusted the rule for file naming not variable naming

@DhairyaMajmudar
Copy link
Copy Markdown
Contributor Author

check out the link in the original issue, you adjusted the rule for file naming not variable naming

oh! true, I've added some rules to fix that

Comment thread biome.json Outdated
Comment thread biome.json Outdated
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.

see suggestions, then good to go

DhairyaMajmudar and others added 3 commits October 7, 2024 04:41
Co-authored-by: Severin Ibarluzea <seve700@gmail.com>
Co-authored-by: Severin Ibarluzea <seve700@gmail.com>
@seveibar seveibar merged commit 32b3ca6 into tscircuit:main Oct 6, 2024
@DhairyaMajmudar DhairyaMajmudar deleted the casing branch October 7, 2024 13:06
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