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

[TC] merge Question. #612

Closed
3 tasks done
Krutok opened this issue Aug 22, 2023 · 9 comments
Closed
3 tasks done

[TC] merge Question. #612

Krutok opened this issue Aug 22, 2023 · 9 comments
Labels
enhancement New feature or request question This issue is in fact a question

Comments

@Krutok
Copy link

Krutok commented Aug 22, 2023

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

  • I understand that my issue may get closed without notice if I intentionally remove or skip any mandatory* field
  • This is NOT a bug/crash report
  • I've searched the bugtracker for similar questions/requests including closed ones

Description

I wanted to ask what the update interval is now. When you release a new NPCBOT patch, will it also be available in your TC fork immediately or with a week delay?

And how many days weeks ago is your fork compared to TC?
I ask because I may plan to take the fork from here again, which then already contains the patch.

TrinityCore or AzerothCore

TrinityCore

Core rev. hash/commit

No response

Operating system

Debian 11

@Krutok Krutok added enhancement New feature or request question This issue is in fact a question labels Aug 22, 2023
@trickerer
Copy link
Owner

As of today (22 Aug 2023) TC+NPCBots repository is up-to-date with TC base 3.3.5 from 20 Aug 2023 (TrinityCore/TrinityCore@1bea4aa)

Each Saturday changes get pulled from upstream (base TC 3.3.5 branch), if merge is successfull then new patch is released the same day, this never changed. Patches get updated using the forks, not vice versa.

Last update was delayed because of major changes made in TC 3.3.5 branch, it took some time to go through bots code and update it.

Also worth mentioning that building TC on Windows now requires Visual Studio 2022 which I don't have on my dev machine currently, so future updates my get delayed if they don't succeed via automerge. But really I don't think there is much to worry about - 3.3.5 TC branch doesn't receive frequent updates anyway.

@Krutok
Copy link
Author

Krutok commented Aug 22, 2023

Thank you for your detailed reply.

@Krutok Krutok closed this as not planned Won't fix, can't repro, duplicate, stale Aug 22, 2023
@Krutok Krutok reopened this Aug 22, 2023
@Krutok Krutok closed this as completed Aug 22, 2023
@trickerer trickerer pinned this issue Aug 24, 2023
@Krutok Krutok reopened this Sep 9, 2023
@Krutok
Copy link
Author

Krutok commented Sep 9, 2023

So your repo hasn't been updated for 2 weeks, although an automatic merge would be possible. And you can't start a PR to start the update process.

Screenshot 2023-09-09 145628

I have now set up my own fork with the last npcbot patch and have installed the last TC update. Everything is compatible, but nothing happens automatically here, although it would be possible.

It's a shame actually.

@trickerer
Copy link
Owner

Well I have no idea why PRs are limited to collaborators, google says GH in some emergency state or something. I don't have any temporary restrictions enabled.

TC automatic merge was failing because I forgot to update automerge repository. Merge is completed now trickerer/TrinityCore-3.3.5-with-NPCBots@a605960.
I'm not updating the patch since I can't run it myself to check if it actually works and won't bug out with basic functionality or even crash at startup.

@Krutok
Copy link
Author

Krutok commented Sep 9, 2023

Well I have no idea why PRs are limited to collaborators, google says GH in some emergency state or something. I don't have any temporary restrictions enabled.

TC automatic merge was failing because I forgot to update automerge repository. Merge is completed now trickerer/TrinityCore-3.3.5-with-NPCBots@a605960. I'm not updating the patch since I can't run it myself to check if it actually works and won't bug out with basic functionality or even crash at startup.

That's why I wanted to help and start a PR, but unfortunately it didn't work.

@Krutok
Copy link
Author

Krutok commented Dec 24, 2023

Is the project dead because there hasn't been an update for months?

@Krutok
Copy link
Author

Krutok commented Jan 31, 2024

It would be very nice if people would comment on whether updates will appear, because the patch is currently no longer compatible with the TC Core

@trickerer
Copy link
Owner

@trickerer
Copy link
Owner

5da5b71

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question This issue is in fact a question
Projects
None yet
Development

No branches or pull requests

2 participants