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

Fix/Fix public-read ACL rule #334

Merged
merged 1 commit into from
Aug 26, 2021
Merged

Fix/Fix public-read ACL rule #334

merged 1 commit into from
Aug 26, 2021

Conversation

carpawell
Copy link
Member

@carpawell carpawell commented Aug 26, 2021

Fix public-read ACL rule, that was same as public-read-write by mistake.

@carpawell carpawell added the bug Something isn't working label Aug 26, 2021
@carpawell carpawell self-assigned this Aug 26, 2021
Signed-off-by: Pavel Karpy <carpawell@nspcc.ru>
@codecov
Copy link

codecov bot commented Aug 26, 2021

Codecov Report

Merging #334 (646d738) into master (8c9e150) will not change coverage.
The diff coverage is n/a.

❗ Current head 646d738 differs from pull request most recent head 3fc2b20. Consider uploading reports for the commit 3fc2b20 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master     #334   +/-   ##
=======================================
  Coverage   62.89%   62.89%           
=======================================
  Files         119      119           
  Lines       13826    13826           
=======================================
  Hits         8696     8696           
  Misses       4138     4138           
  Partials      992      992           

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 8c9e150...3fc2b20. Read the comment docs.

@carpawell carpawell marked this pull request as ready for review August 26, 2021 08:25
@alexvanin alexvanin merged commit 63c3770 into nspcc-dev:master Aug 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants