On 14 Jun 2016 22:30, "Justin W. Flory" <jflory7@gmail.com> wrote:
>
> On 06/02/2016 04:02 PM, James Hogarth wrote:
>>
>>
>>
>> On 1 June 2016 at 18:20, Justin W. Flory <jflory7@gmail.com
>> <mailto:jflory7@gmail.com>> wrote:
>>
>> On 05/31/2016 04:16 AM, James Hogarth wrote:
>>
>> Hi guys,
>>
>> Since jflory liked the look of my blog post for an article in
>> IRC the
>> other day it's now been polished up as a Fedora Magazine article
>> with
>> that audience in mind.
>>
>> It's currently in draft state and can be seen here:
>> https://fedoramagazine.org/?p=13166&preview_id=13166
>>
>> I'm trying to think of what imagery I could add to it but
>> struggling a
>> little in this department. Perhaps pictures of the koji build
>> and bodhi
>> update? or a snippet of the ansible playbook running?
>>
>> I'd be happy to link to the playbooks on github if people think
>> that's
>> sensible (I've removed the links to my blog that discuss how the
>> dynamic
>> inventory works as I suspect my little blog may not survive the
>> traffic
>> directed at it) ...
>>
>> Could you please give it a look over and let me know your
>> thoughts on it.
>>
>> Cheers,
>>
>> James
>>
>>
>> Hey James, I just read over the draft and this looks great to me! I
>> actually learned a lot reading it and I think it should be a good
>> fit for the target audience on the Magazine.
>>
>> I haven't had a chance to review it for edits yet, but I think we
>> have a little bit of time until then to do that. I think we could
>> probably get this one out next week.
>>
>> As for screenshots, I think maybe some of ownCloud might be useful
>> here, such as maybe one of the ownCloud AppStore when you mention
>> it. However, I think the ideas you mentioned would be great for
>> specific to packaging. Koji and Bodhi are good ones to grab, and the
>> Ansible playbook one sounds like a good idea too.
>>
>> We'll make sure we cover this one at tomorrow's meeting. :)
>>
>>
>>
>> Just added some screenshots but my baby befuddled brain is failing to
>> get a nice layout with them ;)
>>
>> I can blame the kid, right?
>>
>
> Hey all,
>
> I just finished up edits on the article and have a final copy pending review here. All it should need is a featured image and maybe one more read-through for any last-minute modifications.
>
> https://fedoramagazine.org/?p=13166&preview=1&_ppp=7819264a2e
>
> Thanks!
>
>

Giving a quick eyes over...

Rather than referring to LetsEncrypt can you please link to the certbot site as that's where the official client now resides and is the upstream of the package:

Rather than the link to max rpm which is quite old now it might be better to link to the Fedora wiki article for that spec file link

https://fedoraproject.org/wiki/How_to_create_a_GNU_Hello_RPM_package

"If they are having issues, it can be nice to offer them a hand in resolving an issue."

Double issue sounds awkward... perhaps resolving the problem is a better phrase?

"A specific process" appears to have lost the anchor to the unresponsive maintainers page

" (such as certbot (formally known as letsencrypt) or sslh) "

The double brackets feels a bit messy... use a pair of commas instead for the LE reference?

"What we do %check in the spec for is if there’s any new bundled libraries (as opposed to just bumped versions). We can automatically load all the PHP libraries correctly."

Last sentence loses context, should be "and that we can automatically load..."

"Thus might include SMB external storage or access to the ownCloud AppStore"

Typo ... thus -> this

________________________

That's all I've got and thank you for the editing and the featured image :)