Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

[fix lint warnings: NFTs pallet] fix clippy::missing_errors_doc lint warnings #14648

Merged

Conversation

sacha-l
Copy link
Contributor

@sacha-l sacha-l commented Jul 26, 2023

This PR fixes the lint warnings emitted by the missing_errors_doc lint in the NFTs pallet by adding an # Errors section to the common helper functions.

@sacha-l sacha-l added A0-please_review Pull request needs code review. I6-documentation Documentation needs fixing, improving or augmenting. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit labels Jul 26, 2023
@sacha-l sacha-l requested a review from jsidorenko as a code owner July 26, 2023 15:39
@sacha-l sacha-l requested a review from a team July 26, 2023 15:39
Copy link

@brunopgalvao brunopgalvao left a comment

Choose a reason for hiding this comment

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

Polishing up the docs on the NFTs pallet nice!

frame/nfts/src/features/attributes.rs Outdated Show resolved Hide resolved
frame/nfts/src/features/attributes.rs Outdated Show resolved Hide resolved
frame/nfts/src/features/attributes.rs Outdated Show resolved Hide resolved
frame/nfts/src/features/metadata.rs Outdated Show resolved Hide resolved
Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
@sacha-l
Copy link
Contributor Author

sacha-l commented Aug 15, 2023

bot merge

@paritytech-processbot paritytech-processbot bot merged commit 4659f8d into master Aug 15, 2023
44 checks passed
@paritytech-processbot paritytech-processbot bot deleted the sl/nfts-fix-missing_errors_doc-lint-warnings branch August 15, 2023 10:00
Ank4n pushed a commit that referenced this pull request Aug 20, 2023
…warnings (#14648)

* fix missing errors doc warnings

* cargo +nightly fmt

* Update frame/nfts/src/features/create_delete_item.rs

* Update frame/nfts/src/features/create_delete_item.rs

* Update frame/nfts/src/features/transfer.rs

* Update frame/nfts/src/features/create_delete_collection.rs

* add intra doc linking for errors

* fmt

* Apply suggestions from code review

Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

---------

Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit I6-documentation Documentation needs fixing, improving or augmenting.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants