Remove source_code.md #1348

jonah · 2019-09-25T12:05:58Z

dawidpotocki commented

2019-09-25 12:05:58 +00:00

(Migrated from github.com)

This file seems useless to me. We already give link to the source code in
the cardv2 recommendations. A lot of repositories are wrong in the
source_code.md, we still list "Secure Hosting Provider" as [WIP], some
sections are outdated like "PC Operating Systems" since it lists Trisquel as
main recommendation and doesn't have Fedora, in some places we give all
recommendations listed and put "closed-source" to closed onces, but in some
we just skip them, formatting is a big mess and… that file actually is not
accessible from the privacytools.io, even though it has a Jekyll front
matter and is accessible from local build on my machine.

So yeah, nobody gives a fsck about that file and just makes maintaining site
a bit more… annoying ;p.

This file seems useless to me. We already give link to the source code in the cardv2 recommendations. A lot of repositories are wrong in the `source_code.md`, we still list "Secure Hosting Provider" as [WIP], some sections are outdated like "PC Operating Systems" since it lists Trisquel as main recommendation and doesn't have Fedora, in some places we give all recommendations listed and put "closed-source" to closed onces, but in some we just skip them, formatting is a big mess and… that file actually is not accessible from the privacytools.io, even though it has a Jekyll front matter and is accessible from local build on my machine. So yeah, nobody gives a fsck about that file and just makes maintaining site a bit more… annoying ;p.

privacytoolsIO (Migrated from github.com) reviewed 2019-09-25 12:05:58 +00:00

jonah reviewed 2019-09-25 12:05:58 +00:00

netlify[bot] commented

2019-09-25 12:06:32 +00:00

(Migrated from github.com)

Deploy preview for privacytools-io ready!

Built with commit 03f2235279

https://deploy-preview-1348--privacytools-io.netlify.com

Deploy preview for *privacytools-io* ready! Built with commit 03f2235279bcbf15a961214adc497513b3a35dfc https://deploy-preview-1348--privacytools-io.netlify.com

blacklight447 (Migrated from github.com) approved these changes 2019-09-25 12:59:37 +00:00

blacklight447 (Migrated from github.com) left a comment

Agreed.

nitrohorse (Migrated from github.com) approved these changes 2019-09-25 15:35:36 +00:00

Mikaela commented

2019-09-26 18:00:54 +00:00

(Migrated from github.com)

I would like to keep it as a lot of work went towards it in https://github.com/privacytoolsIO/privacytools.io/pull/707 and it also contains worth mentioning etc.

dawidpotocki commented

2019-09-27 23:54:07 +00:00

(Migrated from github.com)

I would like to keep it as a lot of work went towards it in https://github.com/privacytoolsIO/privacytools.io/pull/707 and it also contains worth mentioning etc.

That a lot of work was put into something, doesn't mean it shouldn't be
removed. If you want, you can add icon next to worth mentioning items,
but source_code.md is just a mess and makes contributing more
annoying.

> I would like to keep it as a lot of work went towards it in https://github.com/privacytoolsIO/privacytools.io/pull/707 and it also contains worth mentioning etc. That a lot of work was put into something, doesn't mean it shouldn't be removed. If you want, you can add icon next to worth mentioning items, but `source_code.md` is just a mess and makes contributing more annoying.

This repo is archived. You cannot comment on pull requests.

No Reviewers

jonah

privacytoolsIO

blacklight447

nitrohorse

1 Participants

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: privacyguides/privacytools.io#1348