Skip to content

perf: add pytest-benchmark and judge whether use regex match in rabc#228

Merged
hsluoyz merged 1 commit into
apache:masterfrom
sheny1xuan:benchmark
Dec 13, 2021
Merged

perf: add pytest-benchmark and judge whether use regex match in rabc#228
hsluoyz merged 1 commit into
apache:masterfrom
sheny1xuan:benchmark

Conversation

@sheny1xuan
Copy link
Copy Markdown
Contributor

@sheny1xuan sheny1xuan commented Dec 9, 2021

Signed-off-by: stonex 1479765922@qq.com

Signed-off-by: stonex <1479765922@qq.com>
@casbin-bot
Copy link
Copy Markdown

@ffyuanda @Zxilly @techoner @elfisworking please review

@hsluoyz
Copy link
Copy Markdown
Member

hsluoyz commented Dec 9, 2021

@abichinger @Abingcbc @techoner plz review

@abichinger
Copy link
Copy Markdown
Contributor

abichinger commented Dec 9, 2021

Looks good.

We can also judge if domain_matching_func should be used.

See my PR: sheny1xuan#1

@hsluoyz
Copy link
Copy Markdown
Member

hsluoyz commented Dec 13, 2021

@sheny1xuan plz fix the errors:

image

@hsluoyz hsluoyz merged commit 252c288 into apache:master Dec 13, 2021
@hsluoyz
Copy link
Copy Markdown
Member

hsluoyz commented Dec 13, 2021

@abichinger based on comment here: sheny1xuan#1 (comment) , can you make PR in this repo? So we can better find cause of CI errors.

github-actions Bot pushed a commit that referenced this pull request Dec 13, 2021
## [1.15.1](v1.15.0...v1.15.1) (2021-12-13)

### Performance Improvements

* add pytest-benchmark and judge whether use regex in rabc ([#228](#228)) ([252c288](252c288))
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 1.15.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Chimou0 pushed a commit to Chimou0/pycasbin that referenced this pull request Jul 17, 2025
Chimou0 pushed a commit to Chimou0/pycasbin that referenced this pull request Jul 17, 2025
## [1.15.1](apache/casbin-pycasbin@v1.15.0...v1.15.1) (2021-12-13)

### Performance Improvements

* add pytest-benchmark and judge whether use regex in rabc ([apache#228](apache#228)) ([252c288](apache@252c288))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants