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

chore(sdk): fix linguist rule to ignore grpc generated files #4470

Merged
merged 1 commit into from Nov 9, 2022

Conversation

raubitsj
Copy link
Member

@raubitsj raubitsj commented Nov 9, 2022

Description

gRPC files did not match wildcard for generated files

Testing

Not tested.

@raubitsj raubitsj marked this pull request as ready for review November 9, 2022 02:48
@github-actions github-actions bot added cc-chore and removed cc-chore labels Nov 9, 2022
@raubitsj raubitsj requested a review from a team November 9, 2022 02:49
@codecov
Copy link

codecov bot commented Nov 9, 2022

Codecov Report

❗ No coverage uploaded for pull request base (main@09fde74). Click here to learn what that means.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #4470   +/-   ##
=======================================
  Coverage        ?   83.05%           
=======================================
  Files           ?      258           
  Lines           ?    32871           
  Branches        ?        0           
=======================================
  Hits            ?    27302           
  Misses          ?     5569           
  Partials        ?        0           
Flag Coverage Δ
functest 56.89% <0.00%> (?)
unittest 73.14% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

@dmitryduev dmitryduev merged commit 63fa971 into main Nov 9, 2022
@dmitryduev dmitryduev deleted the fix-linguist-again branch November 9, 2022 04:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants