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

Rename type Field to lower-case field #3535

Open
Savio-Sou opened this issue Nov 22, 2023 · 0 comments
Open

Rename type Field to lower-case field #3535

Savio-Sou opened this issue Nov 22, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@Savio-Sou
Copy link
Collaborator

Savio-Sou commented Nov 22, 2023

Problem

All our types are in lower case except Field.

Not the end of the world, but simply less intuitive than it should be.

Happy Case

Field -> field

Alternatives Considered

No response

Additional Context

Breaking change

This would be a breaking change, where every Noir dev would have to rename all Fields in their programs.

Rust's practice

  • Lowercase for builtin types
  • Starts capitalized for custom types

Would you like to submit a PR for this Issue?

No

Support Needs

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: 📋 Backlog
Development

No branches or pull requests

1 participant