Skip to content
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

Removing Unity #987

Merged
merged 10 commits into from
Jun 26, 2024
Merged

Removing Unity #987

merged 10 commits into from
Jun 26, 2024

Conversation

HunterBarclay
Copy link
Member

Description

Completely deleting the Unity project, as well as supporting files and components

Removed

  • Removed /engine/
  • Removed /api/
  • Removed /scripts/
  • Removed engine workflows
  • Removed api workflows

Updated

  • Updated workflow names
  • Updated README.md (Temporary)
  • Updated Logo storage location

@HunterBarclay HunterBarclay self-assigned this Jun 23, 2024
@HunterBarclay HunterBarclay requested a review from a team as a code owner June 23, 2024 16:38
@HunterBarclay HunterBarclay changed the base branch from master to dev June 23, 2024 16:58
Copy link
Member

@BrandonPacewic BrandonPacewic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe the init.bat and init.sh scripts at root need to be updated as well.

README.md Outdated Show resolved Hide resolved
.github/workflows/BlackFormat.yml Outdated Show resolved Hide resolved
HunterBarclay and others added 5 commits June 24, 2024 15:46
Co-authored-by: Brandon Pacewic <pacewicbrandon@gmail.com>
Co-authored-by: Brandon Pacewic <pacewicbrandon@gmail.com>
…ne what we are doing with the protocols directory.
@HunterBarclay
Copy link
Member Author

@BrandonPacewic Made the changes plus changing the header for the "other components" section of the README. I'm leaving in the init scripts for now. Once we decide if we're going to use the protocols folder, we'll remove/keep them. For now, I just removed all the setup for things we removed.

Copy link
Collaborator

@a-crowell a-crowell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excited to see this cleaned out. Almost 600,000 lines, wow.

@HunterBarclay HunterBarclay merged commit d9b09db into dev Jun 26, 2024
5 checks passed
@HunterBarclay HunterBarclay deleted the barclah/nuke branch June 26, 2024 17:06
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