Skip to content

Rename project from redis-starter-rust to kafka-starter-rust#3

Closed
jessevanassen wants to merge 1 commit intocodecrafters-io:mainfrom
jessevanassen:rust-project-name
Closed

Rename project from redis-starter-rust to kafka-starter-rust#3
jessevanassen wants to merge 1 commit intocodecrafters-io:mainfrom
jessevanassen:rust-project-name

Conversation

@jessevanassen
Copy link
Copy Markdown

@jessevanassen jessevanassen commented Sep 14, 2024

Hello!

I noticed that the project name for the Rust based exercises used Redis instead of Kafka. This should fix that (although I don't know if any other tooling outside of the repo depends on the project name).

@andy1li
Copy link
Copy Markdown
Member

andy1li commented Sep 17, 2024

@jessevanassen Thanks for the PR! Unfortunately it failed a test (rust).

I fixed it via #4. Here's our workflow if you're interested:

  1. Only files in starter_templates need to be edited. (Other files will be generated in the second step.)
  2. Generate (compile) other files using course-sdk:
course-sdk compile
  1. Run course-sdk test (Well, my PR failed test (rust) too.)
  2. The fix:
    a. cargo update
    b. course-sdk compile and course-sdk test more one time.

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.

2 participants