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

feat(Deps/FmtLib): update fmtlib to 7.1.3 #5950

Merged
merged 2 commits into from
May 20, 2021

Conversation

Winfidonarleyan
Copy link
Member

@Winfidonarleyan Winfidonarleyan commented May 19, 2021

Changes Proposed:

  • Update fmtlib to 7.1.3

Tests Performed:

image
image

How to Test the Changes:

  • Just compilte and startup applications

How to Test AzerothCore PRs

When a PR is ready to be tested, it will be marked as [WAITING TO BE TESTED].

You can help by testing PRs and writing your feedback here on the PR's page on GitHub. Follow the instructions here:

http://www.azerothcore.org/wiki/How-to-test-a-PR

REMEMBER: when testing a PR that changes something generic (i.e. a part of code that handles more than one specific thing), the tester should not only check that the PR does its job (e.g. fixing spell XXX) but especially check that the PR does not cause any regression (i.e. introducing new bugs).

For example: if a PR fixes spell X by changing a part of code that handles spells X, Y, and Z, we should not only test X, but we should test Y and Z as well.

@Helias
Copy link
Member

Helias commented May 19, 2021

if this new version has no breaking changes I think that is safe to merge

@Winfidonarleyan
Copy link
Member Author

if this new version has no breaking changes I think that is safe to merge

Yes, can merge

@FrancescoBorzi FrancescoBorzi merged commit d86a7b9 into azerothcore:master May 20, 2021
@Winfidonarleyan Winfidonarleyan deleted the lib-fmt branch May 20, 2021 16:57
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.

3 participants