Thanks! You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/991
-- Commit Summary --
* Fix typos in plugin documentation
-- File Changes --
M doc/plugins.dox (20) M doc/pluginsignals.c (8) M doc/pluginsymbols.c (2)
-- Patch Links --
https://github.com/geany/geany/pull/991.patch https://github.com/geany/geany/pull/991.diff
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/991
@@ -98,7 +98,7 @@ at https://github.com/geany/geany-plugins. Beside of adding a new plugin, geany-devel-mailing list is also the place where
Doesn't need the "where"
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/991/files/e222f9e14f9fba5efd5148afb17a3e...
@@ -98,7 +98,7 @@ at https://github.com/geany/geany-plugins. Beside of adding a new plugin, geany-devel-mailing list is also the place where to discuss development related questions. However, once you have done your fork of geany-plugins you can develop -your plugin until you think its the right time to publish it. At this point, +your plugin until you think it's the right time to publish it. At this point,
personally I would prefer "it is"
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/991/files/e222f9e14f9fba5efd5148afb17a3e...
One grammar I noticed while reading and one comment, rest LGTM.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/991#issuecomment-207867169
The commit was updated to include both suggestions.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/991#issuecomment-208046078
Merged #991.
--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/991#event-621717978
github-comments@lists.geany.org