Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fixed manual to match current Sanic app name policy #2461

Merged
merged 3 commits into from May 26, 2022
Merged

fixed manual to match current Sanic app name policy #2461

merged 3 commits into from May 26, 2022

Commits on May 23, 2022

  1. changed app name to match sanic requirements 

    It seems that Sanic has a pretty strict naming policy which doesn't allow spaces and/or comma or similar characters in the app name

following is the exception raised when using the current name as appears in the manual:
`sanic.exceptions.SanicException: Sanic instance named 'Hello World App' uses an invalid format. Names must begin with a character and may only contain alphanumeric characters, _, or -. `
    @amitay87
    amitay87 committed May 23, 2022
    Copy the full SHA
    9add7f4 View commit details
    Browse the repository at this point in the history

  2. changed app name to a more readable form 

    now the name is both valid and readable
    @amitay87
    amitay87 committed May 23, 2022
    Copy the full SHA
    56a97b1 View commit details
    Browse the repository at this point in the history

Commits on May 24, 2022

  1. Merge branch 'main' into main

    @ahopkins
    ahopkins committed May 24, 2022
    Copy the full SHA
    9fb3183 View commit details
    Browse the repository at this point in the history