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

Removed redundant "git-subrepo" folder #10

Closed
wants to merge 2 commits into from
Closed

Removed redundant "git-subrepo" folder #10

wants to merge 2 commits into from

Conversation

JackRayfish
Copy link

Removed the redundant /deps/git-subrepo folder.

I have omitted the pull request template due the simplicity of this change.

Removed the empty and redundant folder "/deps/git-subrepo".
Updated the db_characters SQL files relating to NPC Bots.

- Updated charset to utf8mb4.
- Added collate utf8mb_unicode_ci.
- Added comments.
- Replaced /custom SQL files with top level SQL files.
@trickerer
Copy link
Owner

There are a few problems with these changes.

  1. Deleting trash folder git-subrepo is fine. I wonder where it came from
  2. Please don't tinker with SQL files, I have to keep them in sync across all 3 repos
  3. creature_template_outfits.sql is a part of a different mod integrated into NPCBots and I'd prefer to keep it named as such

I don't ask you to roll back your changes as I understand you share the work you made in your fork but I appreciate the suggestions anyway, I'll pick them manually, hope you don't mind. Closing

@trickerer trickerer closed this Aug 13, 2024
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.

2 participants