Skip to content

[loguru] bumped to 2.1.0#9758

Closed
BullyWiiPlaza wants to merge 1 commit intomicrosoft:masterfrom
BullyWiiPlaza:master
Closed

[loguru] bumped to 2.1.0#9758
BullyWiiPlaza wants to merge 1 commit intomicrosoft:masterfrom
BullyWiiPlaza:master

Conversation

@BullyWiiPlaza
Copy link
Contributor

  • What does your PR fix?
    I had trouble successfully using the library before the update since the loguru.cpp file was missing which needs to be included once somewhere in the code base (counterintuitively). After that, the loguru.hpp can be included as often as one wants.

  • Which triplets are supported/not supported? Have you updated the CI baseline?
    I only bumped up the version.

  • Does your PR follow the maintainer guide?
    Yes.

@msftclas
Copy link

msftclas commented Jan 20, 2020

CLA assistant check
All CLA requirements met.

@PhoebeHui PhoebeHui self-assigned this Jan 20, 2020
@PhoebeHui
Copy link
Contributor

@BullyWiiPlaza, thanks for the PR!

There is a exist PR #8682 to upgrade loguru to 2.1.0.

@BullyWiiPlaza
Copy link
Contributor Author

BullyWiiPlaza commented Jan 20, 2020

@PhoebeHui Oh, excuse me. I searched for loguru but your PR didn't show up before. It's been a while since you submitted your PR so I'm wondering why it takes so long to accept PRs like these? It would be crucial for the package manager to somewhat stay up-to-date.

@PhoebeHui
Copy link
Contributor

@BullyWiiPlaza, the CI testing system offline for maintenance previously that blocked that PR for a long time, we will priority to merge that PR!

@PhoebeHui
Copy link
Contributor

@BullyWiiPlaza, the PR #8682 has been merged. Is that ok to close this PR?

@BullyWiiPlaza
Copy link
Contributor Author

@PhoebeHui Yes, thank you.

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