pull_requests: 1268138688
This data as json
id | node_id | number | state | locked | title | user | body | created_at | updated_at | closed_at | merged_at | merge_commit_sha | assignee | milestone | draft | head | base | author_association | auto_merge | repo | url | merged_by |
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
1268138688 | PR_kwDOAMm_X85LlkLA | 7595 | closed | 0 | Clarifications in contributors guide | 35968931 | Add suggestions @paigem made in #7439, as well as fix a few small formatting things and broken links. I would like to merge this so that it can be helpful for the new contributors we will hopefully get through Outreachy. - [x] Closes #7439 - [ ] ~~Tests added~~ - [x] User visible changes (including notable bug fixes) are documented in `whats-new.rst` - [ ] ~~New functions/methods are listed in `api.rst`~~ | 2023-03-08T16:35:45Z | 2023-03-13T17:55:43Z | 2023-03-13T17:51:24Z | 2023-03-13T17:51:24Z | 4faecb453d414b829803abeda4ba1bedd8142f5d | 0 | ad84fb98c9bb6e068aff4d9ffa93de6d8c4b67dc | 485f801d4bbfc8b2b8c20a390a859ae6cd71ba16 | MEMBER | { "enabled_by": { "login": "TomNicholas", "id": 35968931, "node_id": "MDQ6VXNlcjM1OTY4OTMx", "avatar_url": "https://avatars.githubusercontent.com/u/35968931?v=4", "gravatar_id": "", "url": "https://api.github.com/users/TomNicholas", "html_url": "https://github.com/TomNicholas", "followers_url": "https://api.github.com/users/TomNicholas/followers", "following_url": "https://api.github.com/users/TomNicholas/following{/other_user}", "gists_url": "https://api.github.com/users/TomNicholas/gists{/gist_id}", "starred_url": "https://api.github.com/users/TomNicholas/starred{/owner}{/repo}", "subscriptions_url": "https://api.github.com/users/TomNicholas/subscriptions", "organizations_url": "https://api.github.com/users/TomNicholas/orgs", "repos_url": "https://api.github.com/users/TomNicholas/repos", "events_url": "https://api.github.com/users/TomNicholas/events{/privacy}", "received_events_url": "https://api.github.com/users/TomNicholas/received_events", "type": "User", "site_admin": false }, "merge_method": "squash", "commit_title": "Clarifications in contributors guide (#7595)", "commit_message": "* update links to issue labels\r\n\r\n* clarify that we actually use ruff now in pre-commit\r\n\r\n* mention absolufy-imports which we use in pre-commit\r\n\r\n* checked that example test file works\r\n\r\n* fix formatting of nested bullets\r\n\r\n* add pre-commit to docs env\r\n\r\n* clarify which environment to use for docstrings\r\n\r\n* explain how to view local html docs build\r\n\r\n* clarify what pip -e does\r\n\r\n* update git commit message requirements\r\n\r\n* clarify that small changes don't require local testing\r\n\r\n* whatsnew\r\n\r\n* point to CI build of the docs\r\n\r\n* add suggestion to use github desktop\r\n\r\n* image to help find view docs button on readthedocs\r\n\r\n* update outdated screenshot of green ci\r\n\r\n* internal link to point to where you can view RTD build\r\n\r\n* Add ruff as separate entry in list of pre-commit checks\r\n\r\nCo-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>\r\n\r\n---------\r\n\r\nCo-authored-by: Deepak Cherian <dcherian@users.noreply.github.com>" } |
13221727 | https://github.com/pydata/xarray/pull/7595 |
Links from other tables
- 4 rows from pull_requests_id in labels_pull_requests