Skip to content
This repository was archived by the owner on Sep 16, 2025. It is now read-only.

Conversation

@sundaram2021
Copy link

Added Docker Setup
New Files

  1. Dockerfile
  2. .dockerignore

Updated ruby version and compatible configurations

@sundaram2021 sundaram2021 deleted the patch/add-docker-setup branch September 15, 2025 04:57
@sundaram2021 sundaram2021 restored the patch/add-docker-setup branch September 15, 2025 04:59
@sundaram2021 sundaram2021 reopened this Sep 15, 2025
@sundaram2021
Copy link
Author

please review this @xtine @nicoleslaw @theresaanna @audiodude
and let me know if there is any changes required

@sundaram2021
Copy link
Author

deployment preview
image

@sundaram2021 sundaram2021 changed the title Patch/add docker setup add docker setup Sep 15, 2025
@audiodude
Copy link
Contributor

audiodude commented Sep 16, 2025

I don't know why I was asked to review this, but here we go.

This looks like a completely superfluous change that doesn't provide any value to the website. Why would someone need to dockerize a static site?

Additionally, it needlessly bumps versions of Bundler and other deps, which doesn't even seem necessary to achieve its stated purpose (creating a Docker setup).

I don't have merge rights on this repository, but to whomever does my advice is: Do not merge.

EDIT: I also like to assume good faith, but it's pretty clear this PR is AI generated. Advice is to close.

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