Skip to content

Revert "01 - Feat/defer exercises" - #103

Merged
zhravan merged 1 commit into
mainfrom
revert-91-feat/defer-exercises
Oct 1, 2025
Merged

Revert "01 - Feat/defer exercises"#103
zhravan merged 1 commit into
mainfrom
revert-91-feat/defer-exercises

Conversation

@zhravan

@zhravan zhravan commented Oct 1, 2025

Copy link
Copy Markdown
Owner

Reverts #91

Summary by CodeRabbit

  • Chores

    • Removed the “Deferred Execution” exercise from the catalog; it is no longer visible or accessible to users.
    • Retired associated templates and solution assets for the removed exercise.
  • Tests

    • Removed tests related to the retired “Deferred Execution” exercise.

@coderabbitai

coderabbitai Bot commented Oct 1, 2025

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

Walkthrough

Removed the 28_defer exercise across the codebase: deleted its catalog entry and removed associated solution, template, tests, and utility files. No new files were added, and no other modules were modified.

Changes

Cohort / File(s) Summary
Catalog entry removal
internal/exercises/catalog.yaml
Deleted the 28_defer exercise entry from the catalog.
Template and tests removal
internal/exercises/templates/28_defer/defer.go, internal/exercises/templates/28_defer/defer_test.go, internal/exercises/templates/28_defer/utils.go
Removed template implementation, corresponding test, and helper utilities for 28_defer.
Solution removal
internal/exercises/solutions/28_defer/defer.go
Deleted the solution implementation for 28_defer.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • 01 - Feat/defer exercises #91 — Introduces the 28_defer exercise (catalog, template, solution, helpers); this PR removes the same components, making them directly related but inverse.

Suggested labels

patch

Poem

I hop through code, where files once lay,
Snipping the defer, tidying the way.
Templates hushed, solutions at rest—
A clean burrow is simply the best.
Thump-thump, commit made; onward I cheer,
Less code to nibble, more carrots this year! 🥕

✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch revert-91-feat/defer-exercises

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 831de96 and 7335f25.

📒 Files selected for processing (5)
  • internal/exercises/catalog.yaml (0 hunks)
  • internal/exercises/solutions/28_defer/defer.go (0 hunks)
  • internal/exercises/templates/28_defer/defer.go (0 hunks)
  • internal/exercises/templates/28_defer/defer_test.go (0 hunks)
  • internal/exercises/templates/28_defer/utils.go (0 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@zhravan
zhravan merged commit e36b2ec into main Oct 1, 2025
@zhravan
zhravan deleted the revert-91-feat/defer-exercises branch October 1, 2025 21:37
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.

1 participant