Skip to content

Conversation

@strickvl
Copy link
Contributor

Summary

  • fix stack command typo in huggingface-sagemaker README
  • correct project folder name in magic-photobooth README
  • drop non-existent yolov5 requirements from sign language project
  • update project table descriptions in root README

Testing

  • ruff check . (fails: unrecognized subcommand or lint issues)

@dagshub
Copy link

dagshub bot commented May 17, 2025

@strickvl strickvl requested a review from Copilot May 17, 2025 16:33
@strickvl strickvl added documentation Improvements or additions to documentation internal labels May 17, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Fix mismatches and typos across project READMEs to ensure correct commands and up-to-date project listings.

  • Remove a non-existent requirements install step in the Sign Language Detection README
  • Correct the clone navigation path in the Magic Photobooth README
  • Fix a typo and subcommand in the Huggingface-Sagemaker stack command
  • Update emojis, category labels, and descriptions in the root README’s project table

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
sign-language-detection-yolov5/README.md Removed pip install -r yolov5/requirements.txt line
magic-photobooth/README.md Changed folder path from flux-dreambooth to magic-photobooth
huggingface-sagemaker/README.md Fixed stack command to zenml stack set hf-sagemaker-local
README.md Updated project table emojis, category labels, and descriptions

@strickvl strickvl merged commit 94c5132 into main May 17, 2025
3 of 4 checks passed
@strickvl strickvl deleted the codex/verify-readme-consistency-across-projects branch May 17, 2025 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

codex documentation Improvements or additions to documentation internal

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants