Skip to content

Commit 5c9b1a9

Browse files
SteveL-MSFTCopilot
andcommitted
Address PR feedback: explicit lifetime and dialect delegation
- Use explicit lifetime on register() for public API clarity, with clippy allow attribute for elidable_lifetime_names - Delegate with_vscode_dialect() to with_vscode_vocabulary() instead of with_vscode_keywords() to match the semantic intent chain Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
1 parent bc89a7f commit 5c9b1a9

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

lib/dsc-lib-jsonschema/src/vscode/keywords/vscode_keyword.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,8 @@ impl VSCodeKeyword {
131131
/// [`VSCodeValidationOptionsExtensions`]: crate::vscode::VSCodeValidationOptionsExtensions
132132
/// [`with_vscode_keyword()`]: crate::vscode::VSCodeValidationOptionsExtensions::with_vscode_keyword
133133
/// [`VSCODE_DIALECT_REGISTRY`]: crate::vscode::VSCODE_DIALECT_REGISTRY
134-
pub fn register(self, options: ValidationOptions<'_>) -> ValidationOptions<'_> {
134+
#[allow(clippy::elidable_lifetime_names)]
135+
pub fn register<'i>(self, options: ValidationOptions<'i>) -> ValidationOptions<'i> {
135136
match self {
136137
Self::AllowComments => options.with_keyword(
137138
AllowCommentsKeyword::KEYWORD_NAME,

lib/dsc-lib-jsonschema/src/vscode/validation_options_extensions.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -261,6 +261,6 @@ impl<'i> VSCodeValidationOptionsExtensions<'i> for ValidationOptions<'i> {
261261
self.with_vscode_keywords()
262262
}
263263
fn with_vscode_dialect(self) -> ValidationOptions<'i> {
264-
self.with_vscode_keywords()
264+
self.with_vscode_vocabulary()
265265
}
266266
}

0 commit comments

Comments
 (0)