-
Notifications
You must be signed in to change notification settings - Fork 10
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
feat(): support block pages by license blacklist #4591
Conversation
Warning There were issues while running some tools. Please review the errors and either fix the tool’s configuration or disable the tool if it’s a critical failure. 🔧 eslint
bricks/e2e/src/list-by-use-brick/index.tsxOops! Something went wrong! :( ESLint: 8.57.1 ESLint couldn't find the config "@next-core/eslint-config-next" to extend from. Please check that the name of the config is correct. The config "@next-core/eslint-config-next" was referenced from the config file in "/.eslintrc". If you still have problems, please stop by https://eslint.org/chat/help to chat with the team. 总览这个拉取请求包含了多个仓库和模块的更改,主要涉及路径和页面阻止(blocked)功能的增强。更改范围广泛,包括运行时、认证、菜单处理、错误节点和国际化等多个方面,旨在提供更细粒度的路径访问控制和状态管理。 变更
Finishing Touches
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 on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 7 out of 18 changed files in this pull request and generated no comments.
Files not reviewed (11)
- packages/brick-container/package.json: Language not supported
- packages/easyops-runtime/package.json: Language not supported
- packages/runtime/package.json: Language not supported
- bricks/e2e/src/list-by-use-brick/index.tsx: Evaluated as low risk
- packages/easyops-runtime/src/auth-v2.ts: Evaluated as low risk
- packages/runtime/src/internal/i18n.ts: Evaluated as low risk
- packages/easyops-runtime/src/menu/interfaces.ts: Evaluated as low risk
- packages/runtime/src/internal/Router.ts: Evaluated as low risk
- etc/runtime.api.md: Evaluated as low risk
- packages/easyops-runtime/src/auth.spec.ts: Evaluated as low risk
- packages/runtime/src/internal/ErrorNode.ts: Evaluated as low risk
b48d71a
to
683a0eb
Compare
next-core Run #10869
Run Properties:
|
Project |
next-core
|
Branch Review |
steve/v3-license-blacklist
|
Run status |
Passed #10869
|
Run duration | 00m 22s |
Commit |
72626d2b96 ℹ️: Merge 60fd41482725e8d6088f80010b38c6e5213f5aa1 into 8e258c1ba4a02a2fb0a374f36c8f...
|
Committer | Shenwei Wang |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
0
|
Pending |
0
|
Skipped |
0
|
Passing |
16
|
View all changes introduced in this branch ↗︎ |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## v3 #4591 +/- ##
==========================================
+ Coverage 95.21% 95.25% +0.04%
==========================================
Files 208 208
Lines 8958 8976 +18
Branches 1711 1717 +6
==========================================
+ Hits 8529 8550 +21
+ Misses 321 319 -2
+ Partials 108 107 -1
|
683a0eb
to
0f84de8
Compare
依赖检查
组件之间的依赖声明,是微服务组件架构下的重要信息,请确保其正确性。
请勾选以下两组选项其中之一:
或者:
提交信息检查
Git 提交信息将决定包的版本发布及自动生成的 CHANGELOG,请检查工作内容与提交信息是否相符,并在以下每组选项中都依次确认。
破坏性变更:
feat
作为提交类型。BREAKING CHANGE: 你的变更说明
。新特性:
feat
作为提交类型。问题修复:
fix
作为提交类型。杂项工作:
即所有对下游使用者无任何影响、且没有必要显示在 CHANGELOG 中的改动,例如修改注释、测试用例、开发文档等:
chore
,docs
,test
等作为提交类型。Summary by CodeRabbit
发布说明
新功能
Bug 修复
国际化
依赖更新
@next-api-sdk/api-gateway-sdk
依赖版本到 1.2.2测试