Skip to content
This repository has been archived by the owner on Oct 12, 2023. It is now read-only.

Clean up the sample flask app and PEP8ify the code to standards #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nnja
Copy link

@nnja nnja commented Sep 12, 2018

Made a few updates:

  • General code clean up
  • Updated variable names to conform to Python standards
  • PEP8ified the code

If the changes look good, the tags will need to be updated to reflect the steps in the associated tutorial.

Includes changes to the corresponding tutorial via MicrosoftDocs/azure-docs#14885

@nnja
Copy link
Author

nnja commented Feb 2, 2019

Hey, @cephalin This PR has been open for a few months now. Would you mind taking a look, or pointing me to the right person to review? Thank you.

@cephalin
Copy link
Contributor

cephalin commented Feb 7, 2019

Hi Nina, since we chatted at that time, we've actually changed the Python tutorial to use Django instead. It's not even using this repo anymore. If you want me to merge it I still can.

@nnja
Copy link
Author

nnja commented Feb 12, 2019

@cephalin If this example isn't being actively used in the docs, what do you think about just removing it all together?

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.

2 participants