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 Pyre path find when init Pysa #772

Closed

Conversation

WangGithubUser
Copy link
Contributor

Pre-submission checklist

  • I've ran the linters locally and fixed lint errors related to the files I modified in this PR. You can install the linters by running pip install -r requirements-dev.txt && pre-commit install
  • pre-commit run

Summary

This PR fixed #710.

Test Plan

Pysa should init with no error message Couldn't infer filter directory, skipping filter import to sapp.

Copy link
Contributor

@arthaud arthaud left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! See my requested changes.

client/commands/initialize_pysa.py Outdated Show resolved Hide resolved
client/find_directories.py Outdated Show resolved Hide resolved
@facebook-github-bot
Copy link
Contributor

@arthaud has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@arthaud merged this pull request in 0070693.

@WangGithubUser WangGithubUser deleted the Fix_pyre_path_find branch August 16, 2023 23:32
@arthaud
Copy link
Contributor

arthaud commented Aug 17, 2023

Thanks for the contribution!

@WangGithubUser WangGithubUser mentioned this pull request Sep 22, 2023
2 tasks
facebook-github-bot pushed a commit that referenced this pull request Oct 6, 2023
Summary:
**Pre-submission checklist**
- [x] I've ran the linters locally and fixed lint errors related to the files I modified in this PR. You can install the linters by running `pip install -r requirements-dev.txt && pre-commit install`
- [x] `pre-commit run`

Sorry about I generated this typo in #772 should be `expected` but not `excepted`.

Pull Request resolved: #796

Test Plan: N/A

Reviewed By: kinto0

Differential Revision: D50021836

Pulled By: inseokhwang

fbshipit-source-id: a0ff5be12d4cb0d4be6e37b8e648de95c7bc9e65
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

pyre pysa-init may be checking wrong path for filters
3 participants