autoscan增加过滤路径功能 #55
Merged
autoscan增加过滤路径功能 #55
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Jzow
approved these changes
Jun 27, 2022
| @@ -7,7 +7,7 @@ struct User { | |||
| age: u16, | |||
| } | |||
|
|
|||
| #[summer_boot::auto_scan] | |||
| #[summer_boot::auto_scan("summer-boot-tests/")] | |||
There was a problem hiding this comment.
我理解这样做的目的，我觉得这个是可选的属性，开发者既可以自己指定路径，也可以用自动扫描，我这里建议是为了后期扩展 可以分 component_scan
There was a problem hiding this comment.
没错，这就是一个可选属性，所以在没有添加括号和里面的路径的时候，就是全局扫描
There was a problem hiding this comment.
如果需要添加，则可以在里面手动添加过滤路径，以逗号隔开，就可以隔离多个路径，
#[summer_boot::auto_scan("a", "b")
There was a problem hiding this comment.
如果需要添加，则可以在里面手动添加过滤路径，以逗号隔开，就可以隔离多个路径，
#[summer_boot::auto_scan("a", "b")
这是个不错的提议，如果auto_scan后期变动频繁，我们可以把它封装新的宏，如果不频繁还是使用ummer_boot::auto_scan + 路径
There was a problem hiding this comment.
上述的功能已经在这个pr里已实现，当前pr下更新如下：
#[summer_boot::auto_scan] #[summer_boot::auto_scan("url")] #[summer_boot::auto_scan("url_a", "url_b")]
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
在目前autoscan基础上，提供了过滤路径或文件的功能，可提供过滤扫描的文件或文件夹