Skip to content

Conversation

@WadeWaldron
Copy link
Contributor

@WadeWaldron WadeWaldron commented Oct 8, 2024

Description

Added a Gitpod configuration to the project.

Checklist

  • Unit tests created/updated for any new code (where applicable).
  • Run all tests with ./build.sh validate.
  • Update the CHANGELOG.md.
  • Update the README.md if necessary.

@WadeWaldron WadeWaldron requested a review from a team as a code owner October 8, 2024 15:21
@confluent-cla-assistant
Copy link

🎉 All Contributor License Agreements have been signed. Ready to merge.
Please push an empty commit if you would like to re-run the checks to verify CLA status for all contributors.

Copy link

@alpinegizmo alpinegizmo left a comment

Choose a reason for hiding this comment

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

I played with this; it seems to work fine.

However, I found myself disoriented for two reasons:

  • I found myself wishing that the README had been brought up automatically
  • Once I read through the README I was surprised it told me about staging and solving the exercises, but nothing about how to actually get started with the exercises. I think we need to add a prominent link to CD in the README, to help out anyone who finds this repo without knowing how it connects to the course.

I think improving the README is unrelated to this PR.

@WadeWaldron
Copy link
Contributor Author

@alpinegizmo I like the idea of opening the README by default. Took a quick look and figured out how to do that. Added that to the Gitpod configuration.

While I was in there, it made sense to add the link to the course, so I did that too.

Copy link

@alpinegizmo alpinegizmo left a comment

Choose a reason for hiding this comment

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

Nice.

@WadeWaldron WadeWaldron merged commit 21f76c9 into main Oct 10, 2024
@WadeWaldron WadeWaldron deleted the gitpod2 branch October 10, 2024 14:01
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