-
Notifications
You must be signed in to change notification settings - Fork 169
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
add rules and templates #1
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved, pending license confirmation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're using Apache for SDKs/Cadence, AGPL for Node software, and unlicense I thought for the different cadence standards (NFT/FT) how did MIT get into the mix?
LICENSE
Outdated
MIT License |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did we add MIT and remove apache?
@laynelafrance you're looking at an outdated commit. MIT was removed in |
* Traits view (#1) * Traits view * remove null check * tidy * Rarities (#3) * added rarity * reworded text * make test * Update contracts/MetadataViews.cdc Co-authored-by: Satyam Agrawal <satyam0499@gmail.com> * fix assertion in Rarities * spacing * tidy Co-authored-by: Bjarte Stien Karlsen <bjarte@bjartek.org> Co-authored-by: Satyam Agrawal <satyam0499@gmail.com> * Update contracts/MetadataViews.cdc Co-authored-by: Joshua Hannan <joshua.hannan@dapperlabs.com> * Feedback - Add a wrapper struct around an array of traits and assign to resolver type - Add test for rarity on a trait - Add test for display on a trait * cleanup * rename traitType to name * add an optional max to rarity * Apply suggestions from code review Co-authored-by: Peter Siemens <peterjsiemens@gmail.com> Co-authored-by: Bjarte Stien Karlsen <bjarte@bjartek.org> Co-authored-by: Satyam Agrawal <satyam0499@gmail.com> Co-authored-by: Joshua Hannan <joshua.hannan@dapperlabs.com> Co-authored-by: Peter Siemens <peterjsiemens@gmail.com>
Added CODEOWNERS, contribution guidelines, code of conduct, license, copyright notice, and issue templates