-
Notifications
You must be signed in to change notification settings - Fork 30
docs: Improve homepage documentation clarity and structure #118
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
base: main
Are you sure you want to change the base?
docs: Improve homepage documentation clarity and structure #118
Conversation
- Reorder sections: Install/Auth before Common Tasks - Remove decorative emojis for better accessibility - Add 'Why use the CLI?' and 'What you can do' sections - Expand 'Common tasks' with practical examples - Add automation examples (GitHub Actions, Azure Pipelines) - Reorganize 'Get help' section with clearer categories - Use sentence case for all headings - Add context to all code blocks - Apply active voice and concise language throughout
PR Review: Homepage Documentation ImprovementsThanks for working on improving the docs structure! ✅ What's Good
Also,I agree with @ayeshurun's inline comments. The path syntax issues need to be fixed before this can be merged. A Few More Notes1. Fabric CLI Path Syntax The CLI uses dot-suffix notation — this is a core design principle per the copilot-instructions.md:
Examples:
2. Local→Remote Copy Not Supported
3. Removed Content Worth Preserving The PR removes these community resources:
Consider adding these back under the "Get help" section. 4. Section Ordering "Common tasks" (complex examples) appears before "Run your first command" (simple RecommendationAddress @ayeshurun's inline comments (they're all correct), consider the additional notes above, and this will be ready to merge! |
…ions, add community resources
|
Thanks for the detailed feedback! I've addressed all the comments:
Ready for another look. |
| pip install ms-fabric-cli | ||
| fab auth login -u ${{ secrets.CLIENT_ID }} -p ${{ secrets.CLIENT_SECRET }} --tenant ${{ secrets.TENANT_ID }} | ||
| fab cp ./artifacts/* /workspaces/Production/lakehouses/Data/Files/ | ||
| fab import Production.Workspace/Data.Lakehouse -i ./artifacts/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| fab import Production.Workspace/Data.Lakehouse -i ./artifacts/ | |
| fab import Production.Workspace/Data.Lakehouse -i ./artifacts/DataToImport.Lakehouse |
Changes
Checklist