bookmarks whether to accept PR? #509
-
I hope show PR: https://github.com/alefragnani/vscode-bookmarks/pull/508 |
Beta Was this translation helpful? Give feedback.
Replies: 1 comment 1 reply
-
Hi @yunying1 , Thanks, but in most cases no, the extension is not accepting PRs. Basically because the Additionally, PRs should refers a previously open/discussed issue, specially if you are adding a new feature. Your PR, has no issue linked, so I ask you to open an issue first, to discuss and gather feedback from other users. Hope this helps |
Beta Was this translation helpful? Give feedback.
Hi @yunying1 ,
Thanks, but in most cases no, the extension is not accepting PRs. Basically because the
vscode-bookmarks-core
repository is private. The most common/possible case for PRs should be translations (package.nls.xx.json
) because you can test it updating the installed extension.Additionally, PRs should refers a previously open/discussed issue, specially if you are adding a new feature. Your PR, has no issue linked, so I ask you to open an issue first, to discuss and gather feedback from other users.
Hope this helps