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.

Sign up for GitHub

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

Jump to bottom

feat: Use named loggers #153

Merged
merged 1 commit into from
May 16, 2021
Merged

feat: Use named loggers #153

merged 1 commit into from
May 16, 2021

Conversation

tl24
Copy link
Contributor

@tl24 tl24 commented May 14, 2021 •
edited by hsluoyz
Loading

Fix: #119

@tl24 tl24 mentioned this pull request May 14, 2021
Closed
@tl24 tl24 changed the title Use named loggers feat: Use named loggers May 14, 2021
@tl24 tl24 force-pushed the set-logger branch 2 times, most recently from 6d4fd38 to e9d70b7 Compare May 14, 2021 13:51
@hsluoyz hsluoyz requested review from leeqvip and Zxilly May 14, 2021 13:51
@hsluoyz
Copy link
Member

hsluoyz commented May 15, 2021

@techoner

@leeqvip leeqvip merged commit 40e7f00 into casbin:master May 16, 2021
@hsluoyz
Copy link
Member

hsluoyz commented May 16, 2021 •
edited
Loading

@Zxilly why is there no semantic-release? Please fix it.

Wait for: https://github.com/casbin/pycasbin/actions/runs/846495648

Why is it always queueing?

image

@Zxilly
Copy link
Contributor

Zxilly commented May 16, 2021

@hsluoyz https://www.githubstatus.com/incidents/zbpwygxwb3gw
It's a GitHub Incident, I can't fix it.

github-actions bot pushed a commit that referenced this pull request May 23, 2021
@semantic-release-bot
chore(release): 1.1.0 [skip ci]
c0fcb22 
# [1.1.0](v1.0.6...v1.1.0) (2021-05-23)

### Bug Fixes

* enforce_ex now works fine when it was disabled ([15f58c9](15f58c9))

### Features

* Use named loggers ([#153](#153)) ([40e7f00](40e7f00))
@hsluoyz
Copy link
Member

hsluoyz commented May 23, 2021

🎉 This PR is included in version 1.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Zxilly Zxilly Zxilly approved these changes

@leeqvip leeqvip Awaiting requested review from leeqvip

Assignees
No one assigned
Labels
released
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Improve logger to meet more scenarios
4 participants
@tl24 @hsluoyz @Zxilly @leeqvip