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

Add environment to publish.yml #70

Merged
merged 3 commits into from
Nov 7, 2024
Merged

Add environment to publish.yml #70

merged 3 commits into from
Nov 7, 2024

Conversation

glenn-jocher
Copy link
Member

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

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Updated the GitHub Actions workflow and version number to streamline the deployment process for the ultralytics/thop project.

πŸ“Š Key Changes

  • GitHub Actions Update: Modified the publish.yml workflow file to include an environment section, which specifies deployment details like environment name and URL.
  • Version Bump: Increased the version number from 2.0.10 to 2.0.11.

🎯 Purpose & Impact

  • Improved Deployment Process: By specifying the environment and deployment URL, the changes enhance clarity and organization in deployment management using GitHub's Deployments tab, making it easier to track releases. 🌐
  • Version Update for Consistency: Incrementing the version number ensures consistency and clarity in tracking changes or updates to the software, indicating minor improvements or fixes. πŸ†™

@UltralyticsAssistant UltralyticsAssistant added the devops GitHub Devops or MLops label Nov 7, 2024
@UltralyticsAssistant
Copy link
Member

πŸ‘‹ Hello @glenn-jocher, thank you for submitting a PR to the ultralytics/thop repository! πŸš€ To ensure smooth integration and review, please follow this checklist:

  • βœ… Purpose & Description: Ensure your PR has a clear purpose. Please provide a detailed PR description and link it to any existing issues if applicable.
  • βœ… Branch Synchronization: Verify your branch is up-to-date with the main branch of ultralytics/thop. If not, please synchronize by using the 'Update branch' button or executing git pull and git merge main on your local machine.
  • βœ… CI Checks: Confirm that all Continuous Integration (CI) checks are successful. If any checks fail, please make the necessary corrections.
  • βœ… Documentation: Update any relevant documentation to reflect your changes for new or modified features.
  • βœ… Testing: Include tests for your modifications, and verify that all unit tests are passing.
  • βœ… Contributor License Agreement (CLA): If this is your first contribution to an Ultralytics repository, please sign the Contributor License Agreement by stating, "I have read the CLA Document and I sign the CLA" in a comment.
  • βœ… Minimal Changes: Ensure changes are minimal and focused on the intended purpose, following the principle: "Hack away the unessential." β€” Bruce Lee

This is an automated response, but rest assured, an Ultralytics engineer will review your PR and provide additional feedback or assistance soon. Your contributions help us improve, and we greatly appreciate it! πŸš€πŸŽ‰

For any questions or further guidance, please consult our Contributing Guide. Thanks for contributing to Ultralytics!

@glenn-jocher glenn-jocher changed the title Update publish.yml Add environment to publish.yml Nov 7, 2024
@glenn-jocher glenn-jocher merged commit b01f82c into main Nov 7, 2024
3 checks passed
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch November 7, 2024 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devops GitHub Devops or MLops
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants