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.

Sign up for GitHub

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

Jump to bottom

Add linting script to CI that checks for basic design rules (snake case rules) #106

Closed
seveibar opened this issue Dec 23, 2024 · 4 comments · Fixed by #108
Closed

Add linting script to CI that checks for basic design rules (snake case rules) #106

seveibar opened this issue Dec 23, 2024 · 4 comments · Fixed by #108
Labels
💎 Bounty 💰 Rewarded

Comments

@seveibar
Copy link
Contributor

seveibar commented Dec 23, 2024 •
edited
Loading

  • camel case in zod objects is not allowed
  • enums inside zod objects should be lower snake_case

This can be created as a simple script

Must run on CI

/bounty $5
@algora-pbc Algora PBC
Copy link

algora-pbc bot commented Dec 23, 2024 •
edited
Loading

💎 $5 bounty • tscircuit

Steps to solve:

  1. Start working: Comment /attempt #106 with your implementation plan
  2. Submit work: Create a pull request including /claim #106 in the PR body to claim the bounty
  3. Receive payment: 100% of the bounty is received 2-5 days post-reward. Make sure you are eligible for payouts

Thank you for contributing to tscircuit/circuit-json!

Add a bounty • Share on socials

Attempt Started (GMT+0) Solution
🟢 @Anshgrover23 Dec 23, 2024, 8:21:12 AM #108

devin-ai-integration bot added a commit that referenced this issue Dec 23, 2024
@devin-ai-integration
feat: add lint for naming design rules (#106)
3535789 
Co-Authored-By: [email protected] <[email protected]>
@Anshgrover23
Copy link
Contributor

Anshgrover23 commented Dec 23, 2024 •
edited by algora-pbc bot
Loading

/attempt #106

Algora profile Completed bounties Tech Active attempts Options
@Anshgrover23    14 tscircuit bounties
+ 3 bounties from 3 projects
TypeScript, JavaScript,
Go & more
Cancel attempt

@Anshgrover23 Anshgrover23 mentioned this issue Dec 23, 2024
Merged
@algora-pbc Algora PBC
Copy link

algora-pbc bot commented Dec 23, 2024 •
edited
Loading

💡 @Anshgrover23 submitted a pull request that claims the bounty. You can visit your bounty board to reward.

@algora-pbc Algora PBC
Copy link

algora-pbc bot commented Dec 23, 2024

🎉🎈 @Anshgrover23 has been awarded $5! 🎈🎊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
💎 Bounty 💰 Rewarded
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add linting script to CI that checks for zod:lint Anshgrover23/circuit-json
2 participants
@seveibar @Anshgrover23