Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

fix loop indents #152

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
bobbyiliev merged 1 commit into bobbyiliev:main from liigo:patch-2
Jul 17, 2024
Merged

fix loop indents #152

bobbyiliev merged 1 commit into bobbyiliev:main from liigo:patch-2
Jul 17, 2024

Conversation

@liigo
Copy link
Contributor

@liigo liigo commented Jul 17, 2024

What type of PR is this? (check all applicable)

  • ♻️ Refactor
  • ✨ Feature
  • 🐛 Bug Fix
  • 👷 Optimization
  • 📝 Documentation Update
  • 🚩 Other

Description

Related Tickets & Documents

Added to documentation?

  • 📜 readme
  • 🙅 no documentation needed

[optional] What gif best describes this PR or how it makes you feel?

Copy link

what-the-diff bot commented Jul 17, 2024

PR Summary

  • Improvement in Code Readability
    The indentation of the continue and echo statements inside the first do..done loop has been added. This change will make the codes easier to read and understand, leading to more efficient debugging and modifications.

  • Improved Readability in Second Loop
    The echo statement inside the second do..done loop has also been indented. This will further enhance the readability of the code, thereby reducing the complexity of understanding the script's flow and facilitating future code maintenance.

Copy link
Owner

@bobbyiliev bobbyiliev left a comment

Choose a reason for hiding this comment

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

Looks good! Thank you for the PR.

@bobbyiliev bobbyiliev merged commit 9feeccc into bobbyiliev:main Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bobbyiliev bobbyiliev bobbyiliev approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

AltStyle によって変換されたページ (->オリジナル) /