Re: [Github-comments] [geany/geany-plugins] scope: removed unnecessary includes of 'vte/vte.h' (#793)

2018-11-26 Thread Colomban Wendling
Merged #793 into master.

-- 
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-plugins/pull/793#event-1988745184

Re: [Github-comments] [geany/geany-plugins] scope: removed unnecessary includes of 'vte/vte.h' (#793)

2018-11-26 Thread Colomban Wendling
b4n approved this pull request.

LGTM and it builds on Linux apparently



-- 
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-plugins/pull/793#pullrequestreview-178625809

[Github-comments] [geany/geany-plugins] scope: removed unnecessary includes of 'vte/vte.h' (#793)

2018-11-26 Thread LarsGit223
As discussed in PR #787.
You can view, comment on, or merge this pull request online at:

  https://github.com/geany/geany-plugins/pull/793

-- Commit Summary --

  * scope: removed unnecessary includes of 'vte/vte.h'

-- File Changes --

M scope/src/prefs.c (4)
M scope/src/scope.c (2)

-- Patch Links --

https://github.com/geany/geany-plugins/pull/793.patch
https://github.com/geany/geany-plugins/pull/793.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-plugins/pull/793