Notifications You must be signed in to change notification settings Fork 91 Star 280 Additional navigation options Code Issues Pull requests Actions Projects Wiki Security Insights Commit Permalink refactor: d2-crud-x 迁移到本项目一起维护 Browse files Browse the repository at this point ...
Auto reviews are disabled on base/target branches other than the default branch. Please add the base/target branch pattern to the list of additional branches to be reviewed in the settings. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To tr...
So, when you’re using this refactoring approach, be aware that you’re changing the semantics of your code. If you want to maintain the readonly behavior, use a field declaration in place and assign the field using the primary constructor parameter: Copy namespace Example.Worker.Service; ...
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull...
I'm not really sure what that really means here, and similar for the "Replaced by a private lemma" earlier. The idea of deprecation messages is to make it straight-forward for end-users of mathlib to amend their code to the new changes. This proof has also gone from a one-line proof...
⏰ Context from checks skipped due to timeout of 90000ms (1) GitHub Check: build (8) Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out...
In the file `Lib/logging/__init__.py` I found this pattern ifpossible_false:target=possible_falseelse:target=default_value and thought it can be refactorized to target=possible_falseordefault_value in order to reduce source code space and increase human comprehension speed. ...
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.refactor: improve code structure based on codeclimate hints Browse files This only restructures the code to be more readable based on hints from codeclimate analysis: - simplify...
slimmer Search struct - order gone Search true clone Search conditions renamed to Conditions, sqlConditions struct renamed to SqlConditions, so search_test.go could be moved in tests 21.11.2016 slimmer Search struct - notConditions are gone ...
@dsatcher🤔I think if the next step after auditing the code is to be able to be confident with knowing how large the refactoring task will be, then we should probably audit everything at once. I'm basing this on the assumption that our wiki code isn't huge, which perhaps is wrong....