Skip to content

Conversation

@rahul-vyas-dev
Copy link
Contributor


name: Pull Request
about: Submit changes to the Sugar Labs website for review

📝 Description

This pull request adds a new installation guide page for running Sugar on different-different OS.
The content includes clear step-by-step instructions, OS options (Fedora, Debian, Ubuntu), and a simple layout matching the current website design.
It improves the documentation for Raspberry Pi users and makes the installation process easier to follow.

🔗 Related Issue

Fixes #445

🔄 Type of Change

  • 📱 New Feature (new page, component, or functionality)
  • 🎨 UI/UX Update (visual changes, styling improvements)
  • 📖 Content Update (text changes, documentation)
  • 🐛 Bug Fix
  • ⚡ Performance Improvement
  • ♿ Accessibility Enhancement
  • 🔒 Security Update
  • 📦 Dependency Update
  • 🧹 Code Refactoring
  • 🧪 Test Updates

📷 Visual Changes

Screenshots / GIFs

Before

Screen.Recording.2025-11-09.235436.mp4

After

Screen.Recording.2025-11-10.000401.mp4
Screen.Recording.2025-11-10.000457.mp4
Screen.Recording.2025-11-10.000548.mp4

🧪 Testing Performed

📱 Browser Compatibility

  • Chrome (Latest)
  • Firefox (Latest)
  • Edge (Latest)
  • Safari (Not Available for Testing)
  • Mobile Chrome (Android)
  • Mobile Safari (Not Tested Yet)

🖥️ Responsive Design

  • Desktop (1200px+)
  • Tablet (768px - 1199px)
  • Mobile (320px - 767px)

✅ Test Cases

  1. The Raspberry Pi guide page loads without errors.
  2. All links inside the guide work correctly.
  3. Page layout adapts properly on desktop, tablet, and mobile screens.

♿ Accessibility

  • Proper heading hierarchy maintained
  • ARIA labels added where needed
  • Color contrast requirements met
  • Keyboard navigation works correctly
  • Screen reader testing (To be done soon)

📋 PR Checklist

  • My code follows the project's coding style guidelines
  • I have tested these changes locally
  • I have updated the documentation accordingly
  • My changes generate no new warnings or console errors
  • I have added tests that prove my fix/feature works (Not required for this PR)
  • All existing tests pass successfully
  • I have checked for and resolved any merge conflicts
  • I have optimized images/assets (if applicable)
  • I have validated all links are working correctly

💭 Additional Notes

Feel free to suggest improvements or changes to the structure or content.
I am happy to update the page if needed.


📚 Reviewer Resources

Thank you for contributing to the Sugar Labs website! 🎉

@github-actions
Copy link

github-actions bot commented Nov 9, 2025

❌ Checks Failed

Status: 🚫 Not ready to merge

Please fix the following issues before merging:

🔨 Continuous Integration Failed

Issue: The build process failed to complete.

How to fix:

  1. Run npm run build locally to identify the issue
  2. Fix any TypeScript compilation errors
  3. Ensure all dependencies are properly installed
  4. Test your changes before pushing

📝 Code Linting Failed

Issue: Code formatting or style violations detected.

Specific problems:

• Code formatting issues detected

How to fix:

Platform Command Description
🐧 Unix/macOS/Linux npm run format Auto-fix all formatting issues
🪟 Windows npm run format:file <filename> Fix specific files
🔍 Check Only npm run format:check Check formatting without fixing

Need help with linting? Check out the Linting Guide for Windows Users for detailed instructions.


🛠️ Next Steps

  1. Fix the issues mentioned above
  2. Test locally to ensure everything works
  3. Push your fixes to this branch
  4. Wait for re-check - This bot will automatically run again

🤖 This comment will be updated automatically when you push new commits

@rahul-vyas-dev
Copy link
Contributor Author

Hi @sa-fw-an
could you please review my PR when you get a chance?
I noticed that a few checks have failed, including the Prettier formatting check.
If any formatting fixes or other adjustments are needed, I’ll update the PR right away. Thanks!

@github-actions
Copy link

github-actions bot commented Nov 9, 2025

❌ Checks Failed

Status: 🚫 Not ready to merge

Please fix the following issues before merging:

📝 Code Linting Failed

Issue: Code formatting or style violations detected.

Specific problems:

• Code formatting issues detected

How to fix:

Platform Command Description
🐧 Unix/macOS/Linux npm run format Auto-fix all formatting issues
🪟 Windows npm run format:file <filename> Fix specific files
🔍 Check Only npm run format:check Check formatting without fixing

Need help with linting? Check out the Linting Guide for Windows Users for detailed instructions.


🛠️ Next Steps

  1. Fix the issues mentioned above
  2. Test locally to ensure everything works
  3. Push your fixes to this branch
  4. Wait for re-check - This bot will automatically run again

🤖 This comment will be updated automatically when you push new commits

@github-actions
Copy link

🎉 All Checks Passed!

Status: ✅ Ready to merge

✅ Completed Workflows

Workflow Status Details
🔨 Continuous Integration ✅ Passed Build completed successfully
📝 Code Linting ✅ Passed All formatting and style checks passed

🚀 This PR is ready for review and can be safely merged to main branch!

Great work! Your code meets all quality standards. 👏

@rahul-vyas-dev
Copy link
Contributor Author

Hi @sa-fw-an , I’ve submitted the PR and it’s ready for review.
Please take a look whenever you get time. If any changes are needed—I’ll update them right away.

Thanks!

@sa-fw-an sa-fw-an closed this Nov 15, 2025
@rahul-vyas-dev
Copy link
Contributor Author

hey @sa-fw-an
Hi! I noticed that my PR was closed. Could you please let me know the reason or if any issues are there? If any changes are needed, I’m happy to make them and update the PR.

@sa-fw-an
Copy link
Member

@rahul-vyas-dev This issue was fixed by PR #541 . Thanks for contributing :)

@rahul-vyas-dev
Copy link
Contributor Author

@sa-fw-an I noticed that the documentation for Ubuntu, Debian, Fedora, and Raspberry Pi is already added, but Windows and macOS are missing. Can I work on adding the docs for these platforms?

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.

Visual Guide to Installing Sugar on Raspberry Pi

2 participants