Skip to content

Update google_checks.xml for DPE modifications #115

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

Merged
merged 2 commits into from
Mar 24, 2020
Merged

Update google_checks.xml for DPE modifications #115

merged 2 commits into from
Mar 24, 2020

Conversation

lesv
Copy link
Contributor

@lesv lesv commented Mar 20, 2020

No description provided.

@lesv lesv requested a review from averikitsch March 20, 2020 19:35
@@ -20,6 +20,7 @@ limitations under the License.
<suppressions>
<!-- For our tests we do not demand API documentation. -->
<suppress files=".*\.java" checks="Javadoc.*"/>
<suppress files=".*\.java" checks="CustomImportOrder" />
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this also allow any order for the imports as well as the newline?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@codecov
Copy link

codecov bot commented Mar 20, 2020

Codecov Report

Merging #115 into master will not change coverage by %.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##              master      #115   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity         3         3           
===========================================
  Files              1         1           
  Lines              5         5           
===========================================
  Hits               5         5           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ffce31a...8889fb8. Read the comment docs.

* Add Averi
* Remove CustomImportOrder suppresssion
* Fix AppTest import order
* Updates google_checks to have RegexpHeader, severity, SupressWarningsHolder, and SupressCommentFilter
* Fix the METADATA
@lesv lesv changed the title allow newlines in imports Update google_checks.xml for DPE modifications Mar 24, 2020
@lesv lesv merged commit b95bd05 into master Mar 24, 2020
@lesv lesv deleted the release branch March 24, 2020 19:37
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.

3 participants