Skip to content

Add notification service example to Factory Method README#3515

Open
shabinfrancis wants to merge 1 commit into
iluwatar:masterfrom
shabinfrancis:Shabin-Francis
Open

Add notification service example to Factory Method README#3515
shabinfrancis wants to merge 1 commit into
iluwatar:masterfrom
shabinfrancis:Shabin-Francis

Conversation

@shabinfrancis
Copy link
Copy Markdown

Pull Request Template

What does this PR do?

Adds a new real-world example to the Factory Method pattern documentation.
The example uses a notification system to help readers understand how the Factory Method pattern can be used to create different notification services while keeping the client independent of concrete implementations.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 8, 2026

PR Summary

Adds a real-world example to the Factory Method pattern documentation, showing how a notification system can create Email, SMS, or Push notification services via the Factory Method while keeping the client independent of concrete implementations.

Changes

File Summary
factory-method/README.md Inserted a real-world example describing a messaging app that can send Email, SMS, or Push notifications. The Factory Method selects and creates the correct notification service, demonstrating client decoupling from concrete implementations.

autogenerated by presubmit.ai

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

LGTM!

Review Summary

Commits Considered (1)
  • acaf94c: Add notification service example to Factory Method README
Files Processed (1)
  • factory-method/README.md (1 hunk)
Actionable Comments (0)
Skipped Comments (1)
  • factory-method/README.md [34-36]

    readability: "Improve README blockquote formatting and wording"

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.23%. Comparing base (3b9a2df) to head (acaf94c).

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #3515      +/-   ##
============================================
- Coverage     83.30%   83.23%   -0.08%     
+ Complexity     4025     4020       -5     
============================================
  Files          1060     1060              
  Lines         14246    14246              
  Branches        686      686              
============================================
- Hits          11867    11857      -10     
- Misses         2094     2100       +6     
- Partials        285      289       +4     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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