Skip to content
This repository has been archived by the owner on Mar 31, 2022. It is now read-only.

WIP: Building Web App Chapter 1 #35

Open
wants to merge 76 commits into
base: main
Choose a base branch
from

Conversation

thejeshgn
Copy link
Contributor

No description provided.

@netlify
Copy link

netlify bot commented Jan 5, 2022

✔️ Deploy Preview for monschool ready!

🔨 Explore the source changes: 1294b39

🔍 Inspect the deploy log: https://app.netlify.com/sites/monschool/deploys/621efa44de9d340008eb493d

😎 Browse the preview: https://deploy-preview-35--monschool.netlify.app

@anandology
Copy link
Contributor

Looks good to me @thejeshgn. I think we can merge this PR and you can raise a new PR for each chapter.

tusharsadhwani and others added 23 commits February 26, 2022 10:28
Added support for specifying stdin for examples
The example can add .multi-file class and specify multiple files in the code block using the following convention.

    === app.py
    print(open("index.html").read())

    === index.html
    <h1>Hello, world!</h1>
Added support for including multiple files in examples
Added support for strict mode for mypy.
Mypy Primer: fix small problems with chapter 1 and 2
@anandology
Copy link
Contributor

@thejeshgn please rebase against master so that you can see the autopreview.

@thejeshgn
Copy link
Contributor Author

@anandology done

@anandology
Copy link
Contributor

One suggestion about the How to add a break section.

Just adding
in the example isn't giving a clue of how it works.

My suggestion is to move the How to add a break section after How to make text bold, italized or underlined and add line breaks to the same example.

<strong>Text in bold</strong><br>
<u>Text in underlined</u><br>
<em> - Emphasized text </em>

Also the word break is repeated twice in the title of the How to add a break section. It reads as How to add a break break.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants