Skip to content

Conversation

@Xtansia
Copy link
Collaborator

@Xtansia Xtansia commented Sep 22, 2022

Signed-off-by: Thomas Farr [email protected]

Description

Workflows triggered by dependabot run with a restricted readonly github token, as such it does not have permissions to write the clippy-check annotations, as can be seen here: https://github.com/opensearch-project/opensearch-rs/actions/runs/3093397255/jobs/5005721692
https://github.com/opensearch-project/opensearch-rs/actions/runs/3093397255/jobs/5005721692

Package updates should not introduce clippy lints due to not changing code.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@Xtansia Xtansia requested a review from a team as a code owner September 22, 2022 18:25
@codecov-commenter
Copy link

codecov-commenter commented Sep 22, 2022

Codecov Report

Merging #79 (467ece2) into main (11bd69a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #79   +/-   ##
=======================================
  Coverage   42.32%   42.32%           
=======================================
  Files          49       49           
  Lines       27969    27969           
=======================================
  Hits        11837    11837           
  Misses      16132    16132           
Flag Coverage Δ
integration 42.32% <ø> (ø)

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

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@Xtansia Xtansia requested a review from dblock September 26, 2022 20:08
@dblock dblock merged commit e72288e into opensearch-project:main Sep 27, 2022
@Xtansia Xtansia deleted the fix/dont-run-clippy-on-dependabot-pushes branch September 28, 2022 22:44
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.

4 participants