Skip to content

fix: typos in documentation files - #3186

Merged
iluwatar merged 5 commits into
iluwatar:masterfrom
maximevtush:master
Feb 16, 2025
Merged

fix: typos in documentation files#3186
iluwatar merged 5 commits into
iluwatar:masterfrom
maximevtush:master

Conversation

@maximevtush

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Feb 9, 2025

Copy link
Copy Markdown

PR Summary

This PR fixes typos across various documentation files and source code. The caching.urm.puml, update-method.urm.puml, App.java, CachingTest.java, and CannotSubtractException.java files have been updated to correct spelling and naming inconsistencies.

Changes

File Summary
caching/etc/caching.urm.puml Corrected the typo in useCacheAsideStategy() to useCacheAsideStrategy() in the plantuml diagram.
caching/src/main/java/com/iluwatar/caching/App.java Updated useCacheAsideStategy() method name to useCacheAsideStrategy() for consistency.
caching/src/test/java/com/iluwatar/caching/CachingTest.java Fixed the typo in testCacheAsideStategy() method name to testCacheAsideStrategy().
money/src/main/java/com/iluwatar/CannotSubtractException.java Corrected a typo in the Javadoc description, changing diffrent to different.
update-method/etc/update-method.urm.puml Corrected the typo in postition to position in the plantuml diagram.

autogenerated by presubmit.ai

@@ -12,4 +12,4 @@ public CannotSubtractException(String message) {
super(message);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The word "diffrent" is misspelled as "different".

+ App()
+ main(args : String[]) {static}
+ useCacheAsideStategy()
+ useCacheAsideStrategy()

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The word "Stategy" is misspelled as "Strategy".

public void useCacheAsideStrategy() {
LOGGER.info("# CachingPolicy.ASIDE");
appManager.initCachingPolicy(CachingPolicy.ASIDE);
LOGGER.info(appManager.printCacheContent());

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The word "Stategy" is misspelled as "Strategy".

assertNotNull(app);
app.useCacheAsideStategy();
app.useCacheAsideStrategy();
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The word "Stategy" is misspelled as "Strategy".

+ Skeleton(id : int)
+ Skeleton(id : int, postition : int)
+ Skeleton(id : int, position : int)
+ update()

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The word "postition" is misspelled as "position".

@sonarqubecloud

sonarqubecloud Bot commented Feb 9, 2025

Copy link
Copy Markdown

@iluwatar
iluwatar merged commit 2e7eef1 into iluwatar:master Feb 16, 2025
@iluwatar

Copy link
Copy Markdown
Owner

Looks good! Thank you for the contribution 🎉

@all-contributors please add @maximevtush for code

@allcontributors

Copy link
Copy Markdown
Contributor

@iluwatar

I've put up a pull request to add @maximevtush! 🎉

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