Skip to content
This repository was archived by the owner on Apr 23, 2026. It is now read-only.

fix: Adding explanation of CLI behavior and expanding the error message#3306

Merged
mergify[bot] merged 1 commit intoinstructlab:mainfrom
jpodivin:datapath_arg
Apr 21, 2025
Merged

fix: Adding explanation of CLI behavior and expanding the error message#3306
mergify[bot] merged 1 commit intoinstructlab:mainfrom
jpodivin:datapath_arg

Conversation

@jpodivin
Copy link
Copy Markdown
Contributor

@jpodivin jpodivin commented Apr 20, 2025

Existing documentation was amended to point to correct usage of ilab model train command. The help message of CLI argument --data-path was added to explain interaction with --pipeline argument. The error message was rewritten to explain the most likely cause of the problem.

Resolves #2257

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the
    conventional commits.
  • Changelog updated with breaking and/or notable changes for the next minor release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Functional tests have been added, if necessary.
  • E2E Workflow tests have been added, if necessary.

@mergify mergify Bot added the documentation Improvements or additions to documentation label Apr 20, 2025
@mergify mergify Bot added the one-approval PR has one approval from a maintainer label Apr 20, 2025
@mergify mergify Bot merged commit def4630 into instructlab:main Apr 21, 2025
28 checks passed
@mergify mergify Bot removed the one-approval PR has one approval from a maintainer label Apr 21, 2025
@jpodivin jpodivin deleted the datapath_arg branch April 21, 2025 13:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

documentation Improvements or additions to documentation

Projects

None yet

3 participants