DjangoGirls / tutorial

This is a tutorial we are using for Django Girls workshops
http://tutorial.djangogirls.org/
Other
1.53k stars 1.86k forks source link

Restructure tutorial to separate instructions based on operating system #1796

Open amakarudze opened 1 year ago

amakarudze commented 1 year ago

The tutorial now covers installation guides for 5 platforms, namely, RunCode, ChromeBook, Linux, OS X and Windows. Reading through the installation guide while skipping instructions for 4 platforms you're not using has become tedious. It also makes it difficult to add more platforms or add new content as there are many parts of the tutorial to make edits to.

Separating these installation and deployment instructions by platform makes it easy to address gaps in documentation for each platform. It also makes it easy as it removes the need to read NOTE: If you're not using ... in many places. This is a big change and the first restructuring exercise. Further refactoring may be necessary to reduce repetition.

This pull request addresses #1792 and attempts to shorten the tutorial by grouping OS/platform related content together so that, for example, a Windows user only reads Windows content and a OS X reads only OS X content and so on. Where there are no specific requirements necessary for ChromeBook and RunCode setups, their links have been set to point to Linux instructions.

Changes in this pull request:

amakarudze commented 1 year ago

You also reformat sentences to have hard line breaks at 80 chars. That makes the PR very large and will also mean all translations must update a lot. How hard would it be to undo that line break change?

So @das-g suggested the semantic line breaks in #1781 as they would make it easy to comment on one sentence. They are not hard to remove, however, I would need to know whether to keep each sentence separate and have no clear defined line break or adjust to 120 or have no clearly defined line break policy at all @ekohl. What's your take on this?

das-g commented 1 year ago

So @das-g suggested the semantic line breaks in #1781 as they would make it easy to comment on one sentence.

I suggested semantic line breaks

for newly added paragraphs

in https://github.com/DjangoGirls/tutorial/pull/1781#issuecomment-1370298310, by which I meant "for newly added paragraphs only" and for completely new paragraphs, not moved or edited pre-existing ones. Sorry if my phrasing there wasn't clear enough to convey that. We indeed shouldn't mix paragraph reflows (whether to semantic line breaks or any other convention) into PRs that also change something else, as that will obfuscate the diffs and make the changes even harder to review than not having semantic linefeeds to begin with does.)

(We could of course consider converting existing paragraphs to semantic linefeeds in separate pull requests focusing only on that. I hope that Crowdin would usually came up with the same strings when splitting the text into translatable sentences, and would thus either not require re-translation or at least offer the current translation as suggestion.)

elena commented 1 year ago

Hi Anna,

For Hacktoberfest event there are 2 of us working through this pull request (hello https://github.com/jwhhh !).

We want to confirm there aren't that many actual material changes?

It appears as though mainly file are rearranged and there are substantially more lines because an character line limit is now being used.

We've spent some time looking here and our strategy is that we will break it up and look at the source and also look at the published page and see if it seems OK.

It is harder to check if there have been any major instructions changes but if we find any of these we will check them also.

Generally we feel good so far about the update, @jwhhh just observed that perhaps some of the relative paths need to be updated and is looking to commit fixes.

Another option could be to release a version 2 beta kind of thing? But overall this is great work we're hoping that we can approve soon!

amakarudze commented 10 months ago

Hi Anna,

For Hacktoberfest event there are 2 of us working through this pull request (hello https://github.com/jwhhh !).

We want to confirm there aren't that many actual material changes?

It appears as though mainly file are rearranged and there are substantially more lines because an character line limit is now being used.

We've spent some time looking here and our strategy is that we will break it up and look at the source and also look at the published page and see if it seems OK.

It is harder to check if there have been any major instructions changes but if we find any of these we will check them also.

Generally we feel good so far about the update, @jwhhh just observed that perhaps some of the relative paths need to be updated and is looking to commit fixes.

Another option could be to release a version 2 beta kind of thing? But overall this is great work we're hoping that we can approve soon!

Thanks @elena. I think probably having a version 2 beta would work in this case and leave version 1 as is. Let me work on that for now. I just ran a Django Girls workshop and it was sad participants getting errors because they followed instructions not meant for their platform. Made me wish we had this available sooner rather than later.