[Github-comments] [geany/geany] Added option to auto reload files changed on disk (#1246)

elextr notifications at xxxxx
Wed Mar 29 11:16:32 UTC 2017


> Ah, I was not aware resources were slack. I saw all the activity around the release and thought it would be a good idea to remind the maintainers about this pr.

Its no problem to ping PRs, they do sometimes get forgotten.

> If resources continue to be a problem then maybe a change of style is necessary. Perhaps a well-formed pull request could be pushed to master on a weekly or fortnightly basis and reverted if issues are reported

Well, yes and no.  Certainly some lower risk things could be committed sooner, but...

Because its a mostly interactive program, Geany relies on testing by usage, not large amounts of automated testing.  Because of the resource limitations, to get people to test the latest Git, it has to be stable enough that they are confident to use it for significant periods in real work.  There are not enough resources to spend much time doing nothing but testing Geany.

So anything that reduces the reliability of the Git version also reduces the number of people that are willing to test the latest version, becoming a self-defeating exercise.  So there is some caution about committing things without testing first.

A second issue (that I believe will affect this PR) is that the testers are using Geany in their normal configurations and work flows and that may not involve the particular change, few of us use remote file systems though many non-testing users of Geany seem to do so, only a few of us have multiple monitor setups, only a few use bleeding edge distros and so latest versions of GTK and Glib.  So committing a PR doesn't mean it gets tested.  

>From the discussions above it seems likely that most Geany testers will either not want this feature turned on as it doesn't fit their workflow, or even if they turn it on, it won't get used much since their workflow won't trigger it.  So it won't get tested much, and thats why I suggested you could try to find a tester.

-- 
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/1246#issuecomment-290059319
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20170329/b8be37a1/attachment.html>


More information about the Github-comments mailing list