Skip to content
New issue

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

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

Already on GitHub? # to your account

Update links.yml #40

Merged
merged 2 commits into from
Jan 7, 2024
Merged

Update links.yml #40

merged 2 commits into from
Jan 7, 2024

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Jan 7, 2024

Thank you 🙏 for your contribution to Ultralytics 🚀! Your effort in enhancing our repositories is greatly appreciated. To streamline the process and assist us in integrating your Pull Request (PR) effectively, please follow these steps:

  1. Check for Existing Contributions: Before submitting, kindly explore existing PRs to ensure your contribution is unique and complementary.

  2. Link Related Issues: If your PR addresses an open issue, please link it in your submission. This helps us better understand the context and impact of your contribution.

  3. Elaborate Your Changes: Clearly articulate the purpose of your PR. Whether it's a bug fix or a new feature, a detailed description aids in a smoother integration process.

  4. Ultralytics Contributor License Agreement (CLA): To uphold the quality and integrity of our project, we require all contributors to sign the CLA. Please confirm your agreement by commenting below:

    I have read the CLA Document and I hereby sign the CLA

For more detailed guidance and best practices on contributing, refer to our ✅ Contributing Guide. Your adherence to these guidelines ensures a faster and more effective review process.

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Refactoring GitHub action workflow commands for better code readability.

📊 Key Changes

  • Reformatted the lychee link checker command by splitting it into multiple lines.
  • Ensured each option and input file pattern is on a separate line.

🎯 Purpose & Impact

  • Improved Readability: The change enhances the clarity of the GitHub action configuration, making it easier to read and maintain.
  • Ease of Maintenance: With clearer syntax, future updates or troubleshooting efforts will be more straightforward.
  • Minimal User Impact: There is no direct functionality impact on end-users. This update primarily benefits developers and maintainers of the repository.

@glenn-jocher glenn-jocher merged commit a597bd6 into main Jan 7, 2024
2 of 4 checks passed
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch January 7, 2024 01:01
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant