Skip to content

Dockerize Flask Tutorial Application#6077

Closed
Akshay4527 wants to merge 1 commit into
pallets:mainfrom
Akshay4527:feature/dockerize
Closed

Dockerize Flask Tutorial Application#6077
Akshay4527 wants to merge 1 commit into
pallets:mainfrom
Akshay4527:feature/dockerize

Conversation

@Akshay4527

Copy link
Copy Markdown

Summary

  • Added a Dockerfile to containerize the Flask tutorial application.
  • Added a deploy.sh script to automate Docker build and deployment.
  • Verified the application runs successfully inside a Docker container.
  • Initialized the SQLite database using the Flask CLI.

Testing

  • Built the Docker image successfully.
  • Ran the container on port 5000.
  • Verified the application in the browser.

@ThiefMaster

Copy link
Copy Markdown
Member

See my comment here: #6076 (comment)

It is not acceptable that whoever is supervising your internship/course/whatever is making their students dump this noise into other people's repos.

@nasifabdulrahman

Copy link
Copy Markdown

Hi Adrian,

I'm the mentor who assigned this exercise to the interns, and I wanted to reach out personally.

First, I'm sorry for the unnecessary PRs that ended up in the Flask repository. Our intention was to teach them the Git workflow—forking, branching, committing, and creating pull requests—not to create extra work for your team.

I had instructed them to work on their own forks, but it appears there was a misunderstanding and some PRs were opened against the upstream repository instead. I've already asked them to close those PRs and will make sure future assignments use a company-owned repository or explicitly keep all PRs within internal forks.

Thank you for pointing this out. I appreciate the work you and the Flask maintainers put into the project, and I apologize for the inconvenience caused.

Best,
Nasif
Omega Solutions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants