Skip to content

Remove contents in Website/ - #655

Merged
lukhnos merged 1 commit into
masterfrom
dev/delete-stale-content
Jul 8, 2025
Merged

Remove contents in Website/#655
lukhnos merged 1 commit into
masterfrom
dev/delete-stale-content

Conversation

@lukhnos

@lukhnos lukhnos commented Jul 8, 2025

Copy link
Copy Markdown
Contributor

User description

We haven't updated the files there since 2016. Removing the files now to avoid false positives in automated code review comments like this: #653 (comment)


PR Type

Enhancement


Description

  • Remove stale Website directory files

  • Delete build script and plist config

  • Remove CSS styling and markdown pages


Changes walkthrough 📝

Relevant files
Configuration changes
getVersion.bash
Remove version build script                                                           

Website/updates/getVersion.bash

  • Deleted build version script
+0/-4     
Info.plist
Remove update Info.plist                                                                 

Website/updates/Info.plist

  • Deleted update info plist
+0/-14   
Formatting
markdown.css
Delete website CSS file                                                                   

Website/markdown.css

  • Removed website CSS styling
+0/-82   
Documentation
index-source.markdown
Remove main website markdown                                                         

Website/index-source.markdown

  • Deleted main index markdown content
+0/-63   
textpool-source.markdown
Remove text pool markdown                                                               

Website/textpool-source.markdown

  • Deleted text pool source markdown
+0/-44   
Additional files
index.html [link]   

Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • @lukhnos
    lukhnos requested review from Copilot and zonble July 8, 2025 06:55
    @gemini-code-assist

    Copy link
    Copy Markdown

    Note

    Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

    Copilot AI left a comment

    Copy link
    Copy Markdown
    Contributor

    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 removes all legacy Website files that haven’t been updated since 2016 to prevent noise in automated reviews.

    • Deletes outdated website build scripts and configuration
    • Removes old markup content and styling files
    • Cleans up resources to avoid false positives in tooling

    Reviewed Changes

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

    Show a summary per file
    File Description
    Website/updates/getVersion.bash Removed legacy version-fetching script
    Website/updates/Info.plist Deleted outdated update info property list
    Website/textpool-source.markdown Removed historical text pool documentation
    Website/markdown.css Deleted legacy site styling
    Website/index-source.markdown Removed old homepage source markdown
    Comments suppressed due to low confidence (5)

    @github-actions

    github-actions Bot commented Jul 8, 2025

    Copy link
    Copy Markdown

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No major issues detected

    @github-actions

    github-actions Bot commented Jul 8, 2025

    Copy link
    Copy Markdown

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @lukhnos
    lukhnos merged commit 789f48c into master Jul 8, 2025
    8 checks passed
    @lukhnos
    lukhnos deleted the dev/delete-stale-content branch November 14, 2025 05:39
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    3 participants