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

Update all .buckconfig/.gitignore/.watchmanconfig/buck.iml/.pyre_config to ignore .pylsp related folders #37866

Closed
wants to merge 1 commit into from

Conversation

inseokhwang
Copy link

Summary:
We're updating PYLS and Pyre to use .pylsp as isolation-prefix: D46599960, D46602539.

We'd like to add these .pylsp to ignore folders to prevent users from having bunch of generated files listed in ISL.

I went through every ".lsp-buck-out" and looked for references of .lsp and added new entries with .pylsp where necessary. There are few code mentions of .lsp-buck that seems relevant that I'll send out another diff for so that I can add the project owners.

Differential Revision: D46668238

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner fb-exported labels Jun 13, 2023
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D46668238

…ig to ignore .pylsp related folders (facebook#37866)

Summary:
X-link: facebook/litho#951

X-link: facebook/ocamlrep#40

Pull Request resolved: facebook#37866

X-link: pytorch/pytorch#103551

X-link: facebook/buck2-prelude#11

We're updating PYLS and Pyre to use .pylsp as isolation-prefix: D46599960, D46602539.

We'd like to add these .pylsp to ignore folders to prevent users from having bunch of generated files listed in ISL.

I went through every ".lsp-buck-out" and looked for references of .lsp and added new entries with .pylsp where necessary. There are few code mentions of .lsp-buck that seems relevant that I'll send out another diff for so that I can add the project owners.

Differential Revision: D46668238

fbshipit-source-id: aaca374be0bf27faf1d109e8c3a8bba9a1f8b706
inseokhwang pushed a commit to inseokhwang/pytorch that referenced this pull request Jun 13, 2023
…ml/.pyre_config to ignore .pylsp related folders (pytorch#103551)

Summary:
X-link: facebook/litho#951

X-link: facebook/ocamlrep#40

X-link: facebook/react-native#37866

Pull Request resolved: pytorch#103551

X-link: facebook/buck2-prelude#11

We're updating PYLS and Pyre to use .pylsp as isolation-prefix: D46599960, D46602539.

We'd like to add these .pylsp to ignore folders to prevent users from having bunch of generated files listed in ISL.

I went through every ".lsp-buck-out" and looked for references of .lsp and added new entries with .pylsp where necessary. There are few code mentions of .lsp-buck that seems relevant that I'll send out another diff for so that I can add the project owners.

Test Plan: reviewbylookingveryhard

Differential Revision: D46668238

fbshipit-source-id: 9200b2c052d04db6610c033b30b73761b57679d0
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D46668238

inseokhwang pushed a commit to inseokhwang/buck2 that referenced this pull request Jun 13, 2023
…ig to ignore .pylsp related folders

Summary:
X-link: facebook/litho#951

X-link: facebook/ocamlrep#40

X-link: facebook/react-native#37866

X-link: pytorch/pytorch#103551

X-link: facebook/buck2-prelude#11

We're updating PYLS and Pyre to use .pylsp as isolation-prefix: D46599960, D46602539.

We'd like to add these .pylsp to ignore folders to prevent users from having bunch of generated files listed in ISL.

I went through every ".lsp-buck-out" and looked for references of .lsp and added new entries with .pylsp where necessary. There are few code mentions of .lsp-buck that seems relevant that I'll send out another diff for so that I can add the project owners.

Differential Revision: D46668238

fbshipit-source-id: 64e9a34f444223847f9b58ee2c7f0fa925d08073
inseokhwang pushed a commit to inseokhwang/ocamlrep that referenced this pull request Jun 13, 2023
…ig to ignore .pylsp related folders (facebook#40)

Summary:
X-link: facebook/litho#951

Pull Request resolved: facebook#40

X-link: facebook/react-native#37866

X-link: pytorch/pytorch#103551

X-link: facebook/buck2-prelude#11

We're updating PYLS and Pyre to use .pylsp as isolation-prefix: D46599960, D46602539.

We'd like to add these .pylsp to ignore folders to prevent users from having bunch of generated files listed in ISL.

I went through every ".lsp-buck-out" and looked for references of .lsp and added new entries with .pylsp where necessary. There are few code mentions of .lsp-buck that seems relevant that I'll send out another diff for so that I can add the project owners.

Differential Revision: D46668238

fbshipit-source-id: c10f4d8471579a8ad573fe5c8c213c96343d2552
inseokhwang pushed a commit to inseokhwang/buck2-prelude that referenced this pull request Jun 13, 2023
…ig to ignore .pylsp related folders (facebook#11)

Summary:
X-link: facebook/litho#951

X-link: facebook/ocamlrep#40

X-link: facebook/react-native#37866

X-link: pytorch/pytorch#103551

Pull Request resolved: facebook#11

We're updating PYLS and Pyre to use .pylsp as isolation-prefix: D46599960, D46602539.

We'd like to add these .pylsp to ignore folders to prevent users from having bunch of generated files listed in ISL.

I went through every ".lsp-buck-out" and looked for references of .lsp and added new entries with .pylsp where necessary. There are few code mentions of .lsp-buck that seems relevant that I'll send out another diff for so that I can add the project owners.

Differential Revision: D46668238

fbshipit-source-id: 1ac7072c4aa68d26b54fe5e9522a532d77f2b21f
inseokhwang pushed a commit to inseokhwang/litho that referenced this pull request Jun 13, 2023
…ig to ignore .pylsp related folders (facebook#951)

Summary:
Pull Request resolved: facebook#951

X-link: facebook/ocamlrep#40

X-link: facebook/react-native#37866

X-link: pytorch/pytorch#103551

X-link: facebook/buck2-prelude#11

We're updating PYLS and Pyre to use .pylsp as isolation-prefix: D46599960, D46602539.

We'd like to add these .pylsp to ignore folders to prevent users from having bunch of generated files listed in ISL.

I went through every ".lsp-buck-out" and looked for references of .lsp and added new entries with .pylsp where necessary. There are few code mentions of .lsp-buck that seems relevant that I'll send out another diff for so that I can add the project owners.

Differential Revision: D46668238

fbshipit-source-id: a265a3d38990621e6860b61e352df7ad7383dfee
@cortinico
Copy link
Contributor

Closing as abandoned internally

@cortinico cortinico closed this Aug 7, 2023
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. fb-exported p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants