Skip to content

Conversation

@kenenisa
Copy link
Contributor

This PR performs various cleanup tasks to organize the root directory and remove unnecessary files:

  • Scripts reorganization: Moved all shell scripts from root to a new scripts/ folder, and updated all references in documentation, README, and code (e.g., in create-release.sh).
  • Resource relocation: Moved mark.js from root to ora/Resources/ (overwriting the identical existing file).
  • File removal: Removed .git-blame-ignore-revs as it referenced non-existent commits and had no effect.
  • Documentation cleanup: Removed outdated docs/HOSTING_SETUP.md.

These changes improve project structure without affecting functionality.

Copilot AI review requested due to automatic review settings October 29, 2025 16:13
@kenenisa kenenisa requested a review from yonaries as a code owner October 29, 2025 16:13
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

This PR reorganizes build and setup scripts by moving them into a dedicated scripts/ directory for better project structure. It also removes deprecated documentation files and a JavaScript library, while adding new utility scripts for development.

  • Moved all shell scripts from project root to scripts/ directory
  • Added new scripts: xcbuild-debug.sh, upload-dmg.sh, check-security.sh
  • Updated all script references across documentation files (README, CONTRIBUTING, SECURITY)
  • Removed outdated documentation files and mark.js library

Reviewed Changes

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

Show a summary per file
File Description
scripts/xcbuild-debug.sh New debug build script for Xcode
scripts/upload-dmg.sh New script for uploading DMG files to GitHub releases
scripts/setup.sh Moved setup script to scripts/ directory
scripts/setup-sparkle.sh Moved Sparkle setup script to scripts/ directory
scripts/create-release.sh Updated paths to reference scripts in new location
scripts/check-security.sh New script to verify security configuration
scripts/build-release.sh Moved build script to scripts/ directory
mark.js Removed deprecated JavaScript library
docs/README.md Deleted old documentation file
docs/QUICK_START.md Deleted old documentation file
docs/HOSTING_SETUP.md Deleted old documentation file
SECURITY.md Updated script paths to reference scripts/ directory
README.md Updated setup script path
CONTRIBUTING.md Updated script paths throughout
.gitignore Added *.provisionprofile to ignored files
.git-blame-ignore-revs Removed commit hashes

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@yonaries yonaries merged commit bd59208 into main Oct 29, 2025
8 checks passed
@yonaries yonaries deleted the chore/cleanup-root-dir branch October 29, 2025 17:06
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.

3 participants