Skip to content
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

Update README.md #17

Merged
merged 1 commit into from
Nov 18, 2024
Merged

Update README.md #17

merged 1 commit into from
Nov 18, 2024

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Nov 18, 2024

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Minor update to README formatting, specifically removing an unused HTML break tag.

πŸ“Š Key Changes

  • Deleted an extraneous HTML <br> tag from the top of the README file.

🎯 Purpose & Impact

  • Simplification: Streamlines the document by removing unnecessary code, which can improve readability.
  • No Functional Impact: This change is purely cosmetic with no effect on the functionality of the project.

@UltralyticsAssistant UltralyticsAssistant added the documentation Improvements or additions to documentation label Nov 18, 2024
@UltralyticsAssistant
Copy link
Member

πŸ‘‹ Hello @glenn-jocher, thank you for contributing to the ultralytics/ntc repository with your recent update to the README.md file! πŸš€ This is an automated response, but rest assured that an Ultralytics engineer will assist you soon.

Please ensure your pull request meets the following criteria:

  • βœ… State the Intent: Your PR description is clear and offers a succinct overview of the changes made. Be sure to continue linking to any relevant issues for additional context.
  • βœ… Keep in Sync: Make sure your branch is up-to-date with the main branch of ultralytics/ntc. You can use the 'Update branch' button on GitHub or run git pull and git merge main in your local repository.
  • βœ… Pass All Checks: Ensure all Continuous Integration (CI) checks are passing. If some checks do not pass, identify and fix the issues.
  • βœ… Document Changes: While this change affects documentation, confirm that any other potentially impacted documentation is updated accordingly.
  • βœ… Testing: For this type of documentation change, testing may not be relevant, but remember to validate any functional changes with tests if you have other contributions in the future.
  • βœ… CLA Agreement: Verify that you have signed the Contributor License Agreement by adding a comment "I have read the CLA Document and I sign the CLA" if you haven't done so for prior contributions.
  • βœ… Keep it Lean: Your update represents a focused improvement, echoing Bruce Lee’s wisdom: "It is not daily increase but daily decrease, hack away the unessential. The closer to the source, the less wastage there is."

For detailed guidance, you can check our Contributing Guide. Feel free to ask any questions by leaving a comment. Thank you for your contribution to Ultralytics! πŸŽ‰

@glenn-jocher glenn-jocher merged commit 1b9cef7 into main Nov 18, 2024
1 check passed
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch November 18, 2024 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants