Skip to content

refactor: change naming convention#123

Merged
kkebo merged 1 commit into
mainfrom
naming-convention
Sep 2, 2025
Merged

refactor: change naming convention#123
kkebo merged 1 commit into
mainfrom
naming-convention

Conversation

@kkebo
Copy link
Copy Markdown
Owner

@kkebo kkebo commented Sep 1, 2025

somethingIsPresented -> isSomethingPresented

`somethingIsPresented` -> `isSomethingPresented`
@kkebo kkebo self-assigned this Sep 1, 2025
@kkebo kkebo merged commit 41dfa8b into main Sep 2, 2025
5 checks passed
@kkebo kkebo deleted the naming-convention branch September 2, 2025 13:09
@kkebo kkebo added this to the 1.6.0 milestone Sep 3, 2025
Copy link
Copy Markdown
Owner Author

@kkebo kkebo left a comment

Choose a reason for hiding this comment

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

I made a few mistakes. Those should be reverted.

return
}
logger.debug("DNS settings was saved")
//logger.debug("DNS settings was saved")
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

This is my mistake.

return
}
logger.debug("DNS settings was removed")
//logger.debug("DNS settings was removed")
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

This is my mistake.

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