Skip to content

feat: fix missing await in load_increment_filtered_policy() in async_internal_enforcer.py #378

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

Merged
merged 1 commit into from
Apr 21, 2025

Conversation

KoNEW
Copy link
Contributor

@KoNEW KoNEW commented Apr 21, 2025

No description provided.

@CLAassistant
Copy link

CLAassistant commented Apr 21, 2025

CLA assistant check
All committers have signed the CLA.

@KoNEW KoNEW changed the title fix incremental load_increment_filtered_policy fix(adapter): await in load_increment_filtered_policy Apr 21, 2025
@hsluoyz hsluoyz changed the title fix(adapter): await in load_increment_filtered_policy feat: fix missing await in load_increment_filtered_policy() in async_internal_enforcer.py Apr 21, 2025
@hsluoyz hsluoyz merged commit ca26402 into casbin:master Apr 21, 2025
15 of 17 checks passed
github-actions bot pushed a commit that referenced this pull request Apr 21, 2025
# [1.42.0](v1.41.0...v1.42.0) (2025-04-21)

### Features

* fix missing await in load_increment_filtered_policy() in async_internal_enforcer.py ([#378](#378)) ([ca26402](ca26402))
@hsluoyz
Copy link
Member

hsluoyz commented Apr 22, 2025

@KoNEW can you help fix this CI issue? it blocks the version release of this PR

@KoNEW
Copy link
Contributor Author

KoNEW commented Apr 22, 2025

@KoNEW can you help fix this CI issue? it blocks the version release of this PR

I am not very experienced in contributing)) How can I help?

@KoNEW
Copy link
Contributor Author

KoNEW commented Apr 24, 2025

@hsluoyz I have finally signed CLA, but as u already merged commit I can't change message in history to bypass semantic-pr check step. What should I do?)

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