René's URL Explorer Experiment


Title: Feature/250 add pyright by nathanjmcdougall · Pull Request #256 · rstudio/pins-python · GitHub

Open Graph Title: Feature/250 add pyright by nathanjmcdougall · Pull Request #256 · rstudio/pins-python

X Title: Feature/250 add pyright by nathanjmcdougall · Pull Request #256 · rstudio/pins-python

Description: Contribute to rstudio/pins-python development by creating an account on GitHub.

Open Graph Description: To resolve #250. Similar technique used to generate the requirements file as for #254 . I have triaged each of the rules that are disabled; it looks like some of them would be very easy to re-enabl...

X Description: To resolve #250. Similar technique used to generate the requirements file as for #254 . I have triaged each of the rules that are disabled; it looks like some of them would be very easy to re-enabl...

Opengraph URL: https://github.com/rstudio/pins-python/pull/256

X: @github

direct link

Domain: github.com

route-pattern/_view_fragments/voltron/pull_requests/show/:user_id/:repository/:id/pull_request_layout(.:format)
route-controllervoltron_pull_requests_fragments
route-actionpull_request_layout
fetch-noncev2:691f04b0-c549-025b-1ee7-e8e5b1b7dcce
current-catalog-service-hashae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b
request-id8588:3AF9A7:27BC153:3677E8D:6A4EF313
html-safe-noncedc3b536de1c49aea1ec105525c893957c8a4a549922ac08e3cb805f3bc363200
visitor-payloadeyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI4NTg4OjNBRjlBNzoyN0JDMTUzOjM2NzdFOEQ6NkE0RUYzMTMiLCJ2aXNpdG9yX2lkIjoiNzM0NDIwODU4MjYwMTAxMTk4NyIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9
visitor-hmac94cbd91cf2e5102f23ef08eb077a79e4ed81242302d4c05d2cd8f5a4f5bb7993
hovercard-subject-tagpull_request:1977694113
github-keyboard-shortcutsrepository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot
google-site-verificationApib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I
octolytics-urlhttps://collector.github.com/github/collect
analytics-location///voltron/pull_requests_fragments/pull_request_layout
fb:app_id1401488693436528
apple-itunes-appapp-id=1477376905, app-argument=https://github.com/_view_fragments/voltron/pull_requests/show/rstudio/pins-python/256/pull_request_layout
twitter:imagehttps://opengraph.githubassets.com/9a6c287843259955700438f05ae11af99858570af8cd0cbd970d8d1a1a031bb2/rstudio/pins-python/pull/256
twitter:cardsummary_large_image
og:imagehttps://opengraph.githubassets.com/9a6c287843259955700438f05ae11af99858570af8cd0cbd970d8d1a1a031bb2/rstudio/pins-python/pull/256
og:image:altTo resolve #250. Similar technique used to generate the requirements file as for #254 . I have triaged each of the rules that are disabled; it looks like some of them would be very easy to re-enabl...
og:image:width1200
og:image:height600
og:site_nameGitHub
og:typeobject
og:author:usernamenathanjmcdougall
hostnamegithub.com
expected-hostnamegithub.com
Noneb92d11c0aa4a77d54ef4af1078b6a15fb5a70a215b30c4ecf28889d5a8e656d9
turbo-cache-controlno-cache
go-importgithub.com/rstudio/pins-python git https://github.com/rstudio/pins-python.git
octolytics-dimension-user_id513560
octolytics-dimension-user_loginrstudio
octolytics-dimension-repository_id456588340
octolytics-dimension-repository_nworstudio/pins-python
octolytics-dimension-repository_publictrue
octolytics-dimension-repository_is_forkfalse
octolytics-dimension-repository_network_root_id456588340
octolytics-dimension-repository_network_root_nworstudio/pins-python
turbo-body-classeslogged-out env-production page-responsive
disable-turbofalse
browser-stats-urlhttps://api.github.com/_private/browser/stats
browser-errors-urlhttps://api.github.com/_private/browser/errors
release2b8f23afb982271f1b22258a94aede67a6b77760
ui-targetfull
theme-color#1e2327
color-schemelight dark

Links:

Skip to contenthttps://github.com/rstudio/pins-python/pull/256#start-of-content
https://github.com/
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Frstudio%2Fpins-python%2Fpull%2F256
GitHub CopilotWrite better code with AIhttps://github.com/features/copilot
GitHub Copilot appDirect agents from issue to mergehttps://github.com/features/ai/github-app
MCP RegistryNewIntegrate external toolshttps://github.com/mcp
ActionsAutomate any workflowhttps://github.com/features/actions
CodespacesInstant dev environmentshttps://github.com/features/codespaces
IssuesPlan and track workhttps://github.com/features/issues
Code ReviewManage code changeshttps://github.com/features/code-review
GitHub Advanced SecurityFind and fix vulnerabilitieshttps://github.com/security/advanced-security
Code securitySecure your code as you buildhttps://github.com/security/advanced-security/code-security
Secret protectionStop leaks before they starthttps://github.com/security/advanced-security/secret-protection
Why GitHubhttps://github.com/why-github
Documentationhttps://docs.github.com
Bloghttps://github.blog
Changeloghttps://github.blog/changelog
Marketplacehttps://github.com/marketplace
View all featureshttps://github.com/features
Enterpriseshttps://github.com/enterprise
Small and medium teamshttps://github.com/team
Startupshttps://github.com/enterprise/startups
Nonprofitshttps://github.com/solutions/industry/nonprofits
App Modernizationhttps://github.com/solutions/use-case/app-modernization
DevSecOpshttps://github.com/solutions/use-case/devsecops
DevOpshttps://github.com/solutions/use-case/devops
CI/CDhttps://github.com/solutions/use-case/ci-cd
View all use caseshttps://github.com/solutions/use-case
Healthcarehttps://github.com/solutions/industry/healthcare
Financial serviceshttps://github.com/solutions/industry/financial-services
Manufacturinghttps://github.com/solutions/industry/manufacturing
Governmenthttps://github.com/solutions/industry/government
View all industrieshttps://github.com/solutions/industry
View all solutionshttps://github.com/solutions
AIhttps://github.com/resources/articles?topic=ai
Software Developmenthttps://github.com/resources/articles?topic=software-development
DevOpshttps://github.com/resources/articles?topic=devops
Securityhttps://github.com/resources/articles?topic=security
View all topicshttps://github.com/resources/articles
Customer storieshttps://github.com/customer-stories
Events & webinarshttps://github.com/resources/events
Ebooks & reportshttps://github.com/resources/whitepapers
Business insightshttps://github.com/solutions/executive-insights
GitHub Skillshttps://skills.github.com
Documentationhttps://docs.github.com
Customer supporthttps://support.github.com
Community forumhttps://github.com/orgs/community/discussions
Trust centerhttps://github.com/trust-center
Partnershttps://github.com/partners
View all resourceshttps://github.com/resources
GitHub SponsorsFund open source developershttps://github.com/open-source/sponsors
Security Labhttps://securitylab.github.com
Maintainer Communityhttps://maintainers.github.com
Acceleratorhttps://github.com/open-source/accelerator
GitHub Starshttps://stars.github.com
Archive Programhttps://archiveprogram.github.com
Topicshttps://github.com/topics
Trendinghttps://github.com/trending
Collectionshttps://github.com/collections
Enterprise platformAI-powered developer platformhttps://github.com/enterprise
GitHub Advanced SecurityEnterprise-grade security featureshttps://github.com/security/advanced-security
Copilot for BusinessEnterprise-grade AI featureshttps://github.com/features/copilot/copilot-business
Premium SupportEnterprise-grade 24/7 supporthttps://github.com/enterprise/premium-support
Pricinghttps://github.com/pricing
Search syntax tipshttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
documentationhttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Frstudio%2Fpins-python%2Fpull%2F256
Sign up https://github.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E%2Fvoltron%2Fpull_requests_fragments%2Fpull_request_layout&source=header-repo&source_repo=rstudio%2Fpins-python
Reloadhttps://github.com/rstudio/pins-python/pull/256
Reloadhttps://github.com/rstudio/pins-python/pull/256
Reloadhttps://github.com/rstudio/pins-python/pull/256
Please reload this pagehttps://github.com/rstudio/pins-python/pull/256
rstudio https://github.com/rstudio
pins-pythonhttps://github.com/rstudio/pins-python
Notifications https://github.com/login?return_to=%2Frstudio%2Fpins-python
Fork 11 https://github.com/login?return_to=%2Frstudio%2Fpins-python
Star 58 https://github.com/login?return_to=%2Frstudio%2Fpins-python
Code https://github.com/rstudio/pins-python
Issues 59 https://github.com/rstudio/pins-python/issues
Pull requests 7 https://github.com/rstudio/pins-python/pulls
Actions https://github.com/rstudio/pins-python/actions
Projects https://github.com/rstudio/pins-python/projects
Security and quality 0 https://github.com/rstudio/pins-python/security
Insights https://github.com/rstudio/pins-python/pulse
Code https://github.com/rstudio/pins-python
Issues https://github.com/rstudio/pins-python/issues
Pull requests https://github.com/rstudio/pins-python/pulls
Actions https://github.com/rstudio/pins-python/actions
Projects https://github.com/rstudio/pins-python/projects
Security and quality https://github.com/rstudio/pins-python/security
Insights https://github.com/rstudio/pins-python/pulse
isabelizimmhttps://github.com/isabelizimm
rstudio:mainhttps://github.com/rstudio/pins-python/tree/main
nathanjmcdougall:feature/250-add-pyrighthttps://github.com/nathanjmcdougall/pins-python/tree/feature/250-add-pyright
Conversationhttps://github.com/rstudio/pins-python/pull/256
Commits18 (18)https://github.com/rstudio/pins-python/pull/256/commits
Checkshttps://github.com/rstudio/pins-python/pull/256/checks
Files changedhttps://github.com/rstudio/pins-python/pull/256/files
Feature/250 add pyrighthttps://github.com/rstudio/pins-python/pull/256#top
isabelizimmhttps://github.com/isabelizimm
rstudio:mainhttps://github.com/rstudio/pins-python/tree/main
nathanjmcdougall:feature/250-add-pyrighthttps://github.com/nathanjmcdougall/pins-python/tree/feature/250-add-pyright
https://github.com/nathanjmcdougall
nathanjmcdougallhttps://github.com/nathanjmcdougall
Jul 19, 2024https://github.com/rstudio/pins-python/pull/256#issue-2418547398
Please reload this pagehttps://github.com/rstudio/pins-python/pull/256
#250https://github.com/rstudio/pins-python/issues/250
#254https://github.com/rstudio/pins-python/pull/254
Please reload this pagehttps://github.com/rstudio/pins-python/pull/256
nathanjmcdougallhttps://github.com/nathanjmcdougall
July 19, 2024 19:16https://github.com/rstudio/pins-python/pull/256#commits-pushed-0e0ee92
https://github.com/nathanjmcdougall
Add a new check extra including pyright and pre-commit.https://github.com/rstudio/pins-python/pull/256/commits/0e0ee9268faba516bec5fd56be9a4f3ce968ae5c
0e0ee92https://github.com/rstudio/pins-python/pull/256/commits/0e0ee9268faba516bec5fd56be9a4f3ce968ae5c
https://github.com/nathanjmcdougall
Configure pyright to run in CI.https://github.com/rstudio/pins-python/pull/256/commits/4ec28413cdea4ba3d6f09f96d4d541a92b63ef91
4ec2841https://github.com/rstudio/pins-python/pull/256/commits/4ec28413cdea4ba3d6f09f96d4d541a92b63ef91
https://github.com/nathanjmcdougall
Use pyright action v2https://github.com/rstudio/pins-python/pull/256/commits/a3253cc985560437873241d8ca349acfbcd48a41
a3253cchttps://github.com/rstudio/pins-python/pull/256/commits/a3253cc985560437873241d8ca349acfbcd48a41
https://github.com/nathanjmcdougall
Install dependencies before running pyright.https://github.com/rstudio/pins-python/pull/256/commits/0a6061b591a8e079ab214e9ab448fb6cdfbdbb66
0a6061bhttps://github.com/rstudio/pins-python/pull/256/commits/0a6061b591a8e079ab214e9ab448fb6cdfbdbb66
https://github.com/nathanjmcdougall
Fix sync error.https://github.com/rstudio/pins-python/pull/256/commits/4762898638eabf059840731053e3d080b5b44481
4762898https://github.com/rstudio/pins-python/pull/256/commits/4762898638eabf059840731053e3d080b5b44481
https://github.com/nathanjmcdougall
Revert unnecessary bumps in dev.txthttps://github.com/rstudio/pins-python/pull/256/commits/2105df0edfe12261694e323607f616931c63fe9d
2105df0https://github.com/rstudio/pins-python/pull/256/commits/2105df0edfe12261694e323607f616931c63fe9d
https://github.com/nathanjmcdougall
nathanjmcdougallhttps://github.com/nathanjmcdougall
Jul 20, 2024 https://github.com/rstudio/pins-python/pull/256#ref-issue-2420416818
Add rdata as a test dependency #262 https://github.com/rstudio/pins-python/issues/262
nathanjmcdougallhttps://github.com/nathanjmcdougall
July 20, 2024 14:51https://github.com/rstudio/pins-python/pull/256#commits-pushed-e6189d4
https://github.com/nathanjmcdougall
Ignore the tests folder correctly.https://github.com/rstudio/pins-python/pull/256/commits/e6189d415d46834a1e16cce92ae98f8fdc1e6df4
e6189d4https://github.com/rstudio/pins-python/pull/256/commits/e6189d415d46834a1e16cce92ae98f8fdc1e6df4
https://github.com/nathanjmcdougall
Add appdirs stubshttps://github.com/rstudio/pins-python/pull/256/commits/986a98b536c20f0bc43be20a6ecd482f98581da4
986a98bhttps://github.com/rstudio/pins-python/pull/256/commits/986a98b536c20f0bc43be20a6ecd482f98581da4
https://github.com/nathanjmcdougall
Re-enable reportUnusedExpressionhttps://github.com/rstudio/pins-python/pull/256/commits/ebd603e8c707aa731b37b0d8685ab02cc851bbe9
ebd603ehttps://github.com/rstudio/pins-python/pull/256/commits/ebd603e8c707aa731b37b0d8685ab02cc851bbe9
https://github.com/nathanjmcdougall
Add triaging comments for rule disableshttps://github.com/rstudio/pins-python/pull/256/commits/de4646a785d6314249323f900840e57bed7dbd99
de4646ahttps://github.com/rstudio/pins-python/pull/256/commits/de4646a785d6314249323f900840e57bed7dbd99
https://github.com/nathanjmcdougall
Revert unnecessary project.toml formatting changehttps://github.com/rstudio/pins-python/pull/256/commits/e2a0641b04afed5365ac0a1b9da3d443e7b46a32
e2a0641https://github.com/rstudio/pins-python/pull/256/commits/e2a0641b04afed5365ac0a1b9da3d443e7b46a32
https://github.com/nathanjmcdougall
Revert unnecessary indentation in dev.txthttps://github.com/rstudio/pins-python/pull/256/commits/8e34560c96e083bb91122d25390eabe83fefee95
8e34560https://github.com/rstudio/pins-python/pull/256/commits/8e34560c96e083bb91122d25390eabe83fefee95
https://github.com/isabelizimm
Merge branch 'main' into feature/250-add-pyrighthttps://github.com/rstudio/pins-python/pull/256/commits/70f37427adce7ec7af735cf1caffe15930b97f8d
70f3742https://github.com/rstudio/pins-python/pull/256/commits/70f37427adce7ec7af735cf1caffe15930b97f8d
https://github.com/isabelizimm
isabelizimmhttps://github.com/isabelizimm
Jul 23, 2024 https://github.com/rstudio/pins-python/pull/256#pullrequestreview-2195216072
View reviewed changes https://github.com/rstudio/pins-python/pull/256/files/70f37427adce7ec7af735cf1caffe15930b97f8d
pyproject.tomlhttps://github.com/rstudio/pins-python/pull/256/files/70f37427adce7ec7af735cf1caffe15930b97f8d#diff-50c86b7ed8ac2cf95bd48334961bf0530cdc77b5a56f852c5c61b89d735fd711
Please reload this pagehttps://github.com/rstudio/pins-python/pull/256
https://github.com/isabelizimm
isabelizimmhttps://github.com/isabelizimm
Jul 23, 2024 https://github.com/rstudio/pins-python/pull/256#pullrequestreview-2195216205
View reviewed changes https://github.com/rstudio/pins-python/pull/256/files/70f37427adce7ec7af735cf1caffe15930b97f8d
isabelizimmhttps://github.com/isabelizimm
https://github.com/rstudio/pins-python/pull/256#pullrequestreview-2195216205
Learn morehttps://docs.github.com/articles/managing-disruptive-comments/#hiding-a-comment
Please reload this pagehttps://github.com/rstudio/pins-python/pull/256
nathanjmcdougallhttps://github.com/nathanjmcdougall
July 24, 2024 11:01https://github.com/rstudio/pins-python/pull/256#commits-pushed-bbeb3dc
https://github.com/nathanjmcdougall
https://github.com/isabelizimm
Update pyproject.tomlhttps://github.com/rstudio/pins-python/pull/256/commits/bbeb3dc0a8444374efdeab0df6c99ed2614df610
bbeb3dchttps://github.com/rstudio/pins-python/pull/256/commits/bbeb3dc0a8444374efdeab0df6c99ed2614df610
https://github.com/nathanjmcdougall
Move pyright comments to a tracking issue.https://github.com/rstudio/pins-python/pull/256/commits/839f5583662dccb99c0250116b42fc3439b7d120
839f558https://github.com/rstudio/pins-python/pull/256/commits/839f5583662dccb99c0250116b42fc3439b7d120
https://github.com/nathanjmcdougall
Revert removal of config sections in pyproject.toml.https://github.com/rstudio/pins-python/pull/256/commits/39b227b12844bbeb8eaf189cb5dd46a4521574d0
39b227bhttps://github.com/rstudio/pins-python/pull/256/commits/39b227b12844bbeb8eaf189cb5dd46a4521574d0
https://github.com/nathanjmcdougall
nathanjmcdougallhttps://github.com/nathanjmcdougall
Jul 24, 2024 https://github.com/rstudio/pins-python/pull/256#ref-issue-2426357586
reportIncompatibleVariableOverride: Only one change needed in RsConnectFs; protocol: ClassVar[str | tuple[str, ...]] #276 https://github.com/rstudio/pins-python/issues/276
isabelizimmhttps://github.com/isabelizimm
July 25, 2024 01:13https://github.com/rstudio/pins-python/pull/256#commits-pushed-09c2b1d
https://github.com/isabelizimm
Merge branch 'main' into dev-feature-250-add-pyrighthttps://github.com/rstudio/pins-python/pull/256/commits/09c2b1d42dc7f6172dda2ed5988c7220c679c5af
09c2b1dhttps://github.com/rstudio/pins-python/pull/256/commits/09c2b1d42dc7f6172dda2ed5988c7220c679c5af
https://github.com/isabelizimm
add check deps into tomlhttps://github.com/rstudio/pins-python/pull/256/commits/180c2bfa4ef04feaf16f04ad96340fcc557d8e82
180c2bfhttps://github.com/rstudio/pins-python/pull/256/commits/180c2bfa4ef04feaf16f04ad96340fcc557d8e82
https://github.com/isabelizimm
isabelizimmhttps://github.com/isabelizimm
d821b76https://github.com/rstudio/pins-python/commit/d821b7645c62a00f9740d28c9d2ed66e7cd97da8
Jul 25, 2024https://github.com/rstudio/pins-python/pull/256#event-13640503655
https://github.co/hiddenchars
https://github.com/rstudio/pins-python/pull/{{ revealButtonHref }}
Sign up for freehttps://github.com/join?source=comment-repo
Sign in to commenthttps://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Frstudio%2Fpins-python%2Fpull%2F256
https://github.com/isabelizimm
isabelizimm https://github.com/isabelizimm
https://github.com/rstudio/pins-python/pull/256/files/70f37427adce7ec7af735cf1caffe15930b97f8d
Add pyright for development https://github.com/rstudio/pins-python/issues/250
Please reload this pagehttps://github.com/rstudio/pins-python/pull/256
https://github.com/nathanjmcdougall
https://github.com/isabelizimm
https://github.com
Termshttps://docs.github.com/site-policy/github-terms/github-terms-of-service
Privacyhttps://docs.github.com/site-policy/privacy-policies/github-privacy-statement
Securityhttps://github.com/security
Statushttps://www.githubstatus.com/
Communityhttps://github.community/
Docshttps://docs.github.com/
Contacthttps://support.github.com?tags=dotcom-footer

Viewport: width=device-width

URLs of crawlers that visited me.