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

Rollup of 4 Pull requests with new lints #4839

Merged
merged 11 commits into from
Nov 23, 2019

Conversation

flip1995
Copy link
Member

Rollup of pull requests

changelog: add [zst_offset] lint
changelog: New lint: [ifs_same_cond_fn]
cahngelog: Add new lint [large_stack_arrays]
changelog: added lint [tabs_in_doc_comments]

basil-cow and others added 11 commits November 13, 2019 21:44
added documentation

minor style fix

change as to ::from

add ignore to doc

include threshold in lint message/make suggestion more apparent/use Scalar api instead of matching

style fix

shange snippet_opt to snippet
Run ./util/dev

Revert changelog entry

Rename lint to same_functions_in_if_condition and add a doc example

Add testcases with different arg in fn invocation
@flip1995
Copy link
Member Author

@bors r+ p=4

@bors
Copy link
Contributor

bors commented Nov 23, 2019

📌 Commit 7cc8fa2 has been approved by flip1995

@bors
Copy link
Contributor

bors commented Nov 23, 2019

⌛ Testing commit 7cc8fa2 with merge 35a559f...

bors added a commit that referenced this pull request Nov 23, 2019
Rollup of 4 Pull requests with new lints

Rollup of pull requests

- #4816 (New lint: zst_offset)
- #4814 (New lint: Implement ifs_same_cond_fn)
- #4807 (Add `large_stack_arrays` lint)
- #4806 (Issue/4623)

changelog: add [`zst_offset`] lint
changelog: New lint: [`ifs_same_cond_fn`]
cahngelog: Add new lint [large_stack_arrays]
changelog: added lint [`tabs_in_doc_comments`]
@bors
Copy link
Contributor

bors commented Nov 23, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: flip1995
Pushing 35a559f to master...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants