Process site only if not in `watch` mode

[rpeyron]

Fix mentioned in #699

Will avoid duplicate site processing: either we are in watch mode and jekyll-admin do not need to trigger site.process, or we are not in watch mode and jekyll-admin will trigger site.process

Note that it does not work when using --livereload --no-watch options simultaneously, which has little interest, as livereload option will force config{"watch"]=1.

Closes #699

[ashmaroli]

Thank you for submitting this pull request, @rpeyron.

Leaving a few points as feedback:

• The name do_process isn't fully transparent to the underlying implementation. How about conditionally_process_site? That way, a future contributor (including us) will know what to expect immediately on reading the code.
• The Admin interface allows one to edit the configuration file as well. So, if I remember correctly, that may allow changing site.config["watch"] (without actually loading Jekyll's Watcher). I haven't tested this; just a hypothesis. The solution for this could be initializing a new object (e.g. JekyllAdmin.site_processor). If this is too confusing for you, let me know. I will make that change in a separate pull request / commit.

[rpeyron]

Hello @ashmaroli ; thanks for your feedback

• You are right, I have renamed as suggested
• I am not sure to have understood the solution you have in mind, I will leave it to you

[ashmaroli]

Thank you @rpeyron