Skip to content

Handle the case where appid contains at least one upperletter #1233

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

Merged
philliphoff merged 7 commits into from
Feb 16, 2024
Merged

Handle the case where appid contains at least one upperletter #1233

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
d003688
Handle the case where appid can contain some uppercases
TWEESTY Feb 9, 2024
b8d028f
Add one test sample
TWEESTY Feb 9, 2024
41a412f
Optimization in order to not add some overhead time for the "normal" …
TWEESTY Feb 9, 2024
f261800
Change comment which was false
TWEESTY Feb 9, 2024
df02fc2
Remove the breaking change
TWEESTY Feb 10, 2024
dcd9dee
Simplify according to the review
TWEESTY Feb 16, 2024
cfa1bb2
Merge branch 'master' into master
TWEESTY Feb 16, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Merge branch 'master' into master
  • Loading branch information
@TWEESTY
TWEESTY authored Feb 16, 2024
commit cfa1bb2d47d3da7b0004c84a56b8cfa243a4a528

This merge commit was added into this branch cleanly.

There are no new changes to show, but you can still view the diff.