feat(move): Move view functions (compiler implementation) - Linter Warning#11444
Open
lollobene wants to merge 2 commits intovm-lang/8254-move-view-functionsfrom
Open
feat(move): Move view functions (compiler implementation) - Linter Warning#11444lollobene wants to merge 2 commits intovm-lang/8254-move-view-functionsfrom
lollobene wants to merge 2 commits intovm-lang/8254-move-view-functionsfrom
Conversation
05dfebd to
cb583a9
Compare
d05d057 to
c099118
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of change
Linter warning implementation
This change gates the Iota view-function suggestion lint behind
--lint.ViewFunctionVisitoris no longer included in the default Iota linter set, so normal builds do not emit warnings for functions that could be annotated with#[view]. The visitor now only runs atLintLevel::All, which is enabled through the existing--lintflag.Links to any relevant issues
fixes #11413