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 filepathprovider generic type #1036

Merged
merged 2 commits into from
Oct 6, 2023
Merged

Conversation

kinto0
Copy link
Contributor

@kinto0 kinto0 commented Oct 5, 2023

Summary

The file path provider claims it provides a collection of paths but in reality it provides a single path. This is a hole because gen_cache doesn't have great type annotations: example.

Test Plan

CI

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 5, 2023
@codecov-commenter
Copy link

codecov-commenter commented Oct 5, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (7ca5d7f) 91.09% compared to head (445c612) 91.09%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1036   +/-   ##
=======================================
  Coverage   91.09%   91.09%           
=======================================
  Files         255      255           
  Lines       26564    26564           
=======================================
  Hits        24198    24198           
  Misses       2366     2366           
Files Coverage Δ
libcst/metadata/file_path_provider.py 100.00% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@kinto0 kinto0 marked this pull request as ready for review October 6, 2023 00:26
@zsol zsol merged commit 83f0dae into Instagram:main Oct 6, 2023
24 checks passed
@zsol
Copy link
Member

zsol commented Oct 6, 2023

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants