| Age | Commit message (Collapse) | Author |
|---|
|
am: 5a3fcff118
Change-Id: I9c32e7d69425952b41792f61c7f88d29452c26cd
|
|
494e25f7e0 am: 97c2713928
am: f4b59ee7bf
Change-Id: I99b3c7f549f1e22e6bd077dad8c4d1f15412e33c
|
|
am: c547d8f0b0
Change-Id: I9493a8a056c8fd765d9aafc6c2514d7ebcced1b2
|
|
494e25f7e0
am: 97c2713928
Change-Id: I531c98a8bcd6672a08de5dd6ff8ea826c30ac9ac
|
|
am: aa8a88f3a9
Change-Id: I783ae40db4aa9739fd5178c86ccd2130d67cc190
|
|
am: 494e25f7e0
Change-Id: Icef9de74979f70a6b63bcce29e30d2ecaffe9349
|
|
Android prebuilt python stdlib does not include the modules needed to
determine sysconfig values. Since this is only an issue on mac (and
manifests only as a miscount of characters) we will work around this
by simply skipping the test if it fails.
Test: m cpplint-art
Bug: 144866010
Bug: 144931206
Change-Id: I5a4d317e26f8ee4e230d1fc7bbd878e3ca939f63
|
|
Test: manual
Change-Id: I187eb100f4b08f0f8746040612e406dca34981b3
|
|
C++17 adds support for tuple destructuring. This allow one to write
code such as:
```
std::pair<int, int> span = getSpan();
auto [start, end] = span;
// Use start as span.first and end as span.second
```
This makes cpplint recognize and allow a space before the '[' in this
situation.
This is a purposeful divergence from the internal version where the
entire whitespace/braces category was removed. It was decided to leave
the checks in since this is sometimes used without other formatting
tools.
Test: manual
|
|
PiperOrigin-RevId: 273904189
|
|
Spelling mistake
|
|
Minor spelling/grammar correction
|
|
|
|
Minor spelling mistake
|
|
Update C++ style guide for C++17
|
|
|
|
Update C++ style guide
|
|
|
|
Update JS styleguide
Closes #462.
|
|
Also fix a couple of issues.
|
|
|
|
|
|
#459 replaced the R style guide and removed Rguide.xml, but we need this file to prevent broken links.
|
|
Replace external R style guide
|
|
Internally, we've moved to the Tidyverse Style Guide: style.tidyverse.org
The new file for Rguide.md reflects this fact and notes where we differ.
|
|
The HTML Line-wrapping recommendation is to use four additional spaces when wrapping HTML, yet the “Not recommended” examples for the `type` attribute rule were using two spaces.
|
|
update jQuery version in example
|
|
I know it's only an example, but best to have the most recent version here which doesn't have a known security vulneability.
|
|
Removed RAS (redundant acronym syndrome) syndrome
|
|
References to HTTPS and HTTP were written as HTTPS protocol (Hypertext Transfer Protocol Secure protocol) and HTTP protocol (Hypertext Transfer Protocol protocol) which is redundant.
|
|
PiperOrigin-RevId: 224224079
|
|
PiperOrigin-RevId: 223189743
|
|
Github pages sees pyguide.html and renders the markdown via that url when no actual .html file is present in the repo.
|
|
PiperOrigin-RevId: 222009069
|
|
PiperOrigin-RevId: 221622606
|
|
|
|
|
|
[objc] Update the Objective-C style guide 📜
|
|
|
|
Add horizontal TOC back into style guide
|
|
|
|
Update C++ style guide
|
|
|
|
Merge pull request #373 from leVirve/gh-pages.
internally copybara was leaving extra space in a bad spot while stripping a comment. fixed the internal version to not do this and produce the same output this PR has.
|
|
|
|
Link to the maintained pyguide.md, not old html.
|
|
pyguide.html is unmaintained and has been for ~5 years. pyguide.md
is reasonably up to date and we have the ability to keep it up to
date from our internal guide. Link to that instead.
I'd like to delete the old pyguide.html but a lot of things still
point directly at it so they at least have somewhere to land for
the time being.
|
|
|
|
Fix a typo in cpplint
|
|
s/seperated/separated
|
