Skip to content

Make Docker actions conditional on ultralytics/yolov5 repo#8060

Merged
glenn-jocher merged 4 commits intomasterfrom
update/ci
May 31, 2022
Merged

Make Docker actions conditional on ultralytics/yolov5 repo#8060
glenn-jocher merged 4 commits intomasterfrom
update/ci

Conversation

@glenn-jocher
Copy link
Copy Markdown
Member

@glenn-jocher glenn-jocher commented May 31, 2022

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Enhanced CI and Docker workflows for better repository management 🚀

📊 Key Changes

  • Added new environment variable outputs to the CI testing workflow.
  • Commented out the pull request trigger for Docker image publishing.
  • Added a condition to only run the Docker job for the official Ultralytics repository.

🎯 Purpose & Impact

  • 🛠 Improves debugging by echoing additional GitHub context variables in CI.
  • 📝 Prevents unnecessary Docker builds on pull request events, conserving resources.
  • 🛂 Ensures Docker images are only published from the main Ultralytics repository, protecting the integrity of official images and preventing accidental pushes from forks.

@glenn-jocher glenn-jocher changed the title Update ci-testing.yml Make Docker actions conditional on ultralytics/yolov5 repo May 31, 2022
@glenn-jocher glenn-jocher self-assigned this May 31, 2022
@glenn-jocher glenn-jocher merged commit 7d87b9e into master May 31, 2022
@glenn-jocher glenn-jocher deleted the update/ci branch May 31, 2022 14:36
@glenn-jocher
Copy link
Copy Markdown
Member Author

@Borda @AyushExel Docker builds no longer run on forks, thanks for the tip! The action still runs but shows as 'check skipped':

Verified on my personal fork at https://github.com/glenn-jocher/yolov5:
Screenshot 2022-05-31 at 16 43 21

tdhooghe pushed a commit to tdhooghe/yolov5 that referenced this pull request Jun 10, 2022
…tics#8060)

* Update ci-testing.yml

* Update ci-testing.yml

* Update ci-testing.yml

* Update docker.yml
ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this pull request Sep 8, 2022
…tics#8060)

* Update ci-testing.yml

* Update ci-testing.yml

* Update ci-testing.yml

* Update docker.yml
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