Skip to content

Conversation

@leerob
Copy link
Contributor

@leerob leerob commented May 29, 2022

@ijjk ijjk added examples Issue was opened via the examples template. created-by: Next.js team PRs by the Next.js team. labels May 29, 2022
@@ -1,45 +0,0 @@
# gh-pages Hello World example
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to leave this README and have a note linking to the new name similar to https://github.com/vercel/next.js/blob/canary/examples/with-material-ui/README.md

Copy link
Contributor Author

@leerob leerob May 29, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm leaning towards no. Here's why: we have so many examples now, we're almost reaching "folder debt" where I'd rather have a low probability 404 on gh-pages than harder scanning through the directory. I also instinctively searched for "github" and didn't find anything, so I almost missed this example entirely.

Open to other thoughts.

@kodiakhq kodiakhq bot merged commit c4fdfcb into canary May 29, 2022
@kodiakhq kodiakhq bot deleted the github-pages-deployment branch May 29, 2022 20:12
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 29, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

created-by: Next.js team PRs by the Next.js team. examples Issue was opened via the examples template.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants