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

🔧 Setup initial stuff #3

Merged
merged 6 commits into from
Feb 18, 2025
Merged

🔧 Setup initial stuff #3

merged 6 commits into from
Feb 18, 2025

Conversation

TimNekk
Copy link
Member

@TimNekk TimNekk commented Feb 18, 2025

This pull request includes significant changes to the AstroShopping project, focusing on the removal of legacy code, updates to naming conventions, and modifications to configuration files. Below is a summary of the most important changes:

Codebase Simplification:

  • Removed legacy character class AAstroShoppingCharacter and its associated header file AstroShoppingCharacter.h. [1] [2]
  • Removed AstroShoppingGameMode references to the old character class.
  • Removed the AstroShoppingPlayerController class and its header file. [1] [2]
  • Removed the AstroShoppingProjectile class and its header file. [1] [2]
  • Removed the TP_PickUpComponent class.

Configuration Updates:

  • Added naming convention rules and styles for C++ files in .editorconfig, ensuring consistency with Unreal Engine standards.
  • Updated .github/ISSUE_TEMPLATE/bug_report.md and .github/ISSUE_TEMPLATE/feature_request.md to use type instead of labels. [1] [2]

Code Enhancements:

  • Modified DefaultInput.ini to set DefaultTouchInterface to None.

These changes aim to streamline the codebase, enforce naming conventions, and update configurations to align with current project needs.

@TimNekk TimNekk added the feature New feature or request label Feb 18, 2025
@TimNekk TimNekk self-assigned this Feb 18, 2025
@TimNekk TimNekk linked an issue Feb 18, 2025 that may be closed by this pull request
@TimNekk TimNekk merged commit 5ceaeca into develop Feb 18, 2025
@TimNekk TimNekk deleted the 2-setup-initial-stuff branch February 18, 2025 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setup initial stuff
1 participant