Skip to content

Adds experimental flag to the basyx.feature.search (AASQL)#826

Merged
aaronzi merged 1 commit into
eclipse-basyx:mainfrom
aaronzi:main
Aug 22, 2025
Merged

Adds experimental flag to the basyx.feature.search (AASQL)#826
aaronzi merged 1 commit into
eclipse-basyx:mainfrom
aaronzi:main

Conversation

@aaronzi

@aaronzi aaronzi commented Aug 22, 2025

Copy link
Copy Markdown
Member

Description of Changes

This PR marks the search feature (AAS Query Language) as experimental

@aaronzi aaronzi requested a review from Copilot August 22, 2025 15:04

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR marks the BaSyx search feature (AAS Query Language) as experimental by updating configuration properties to include "experimental" in the feature flag hierarchy.

  • Updates all search feature configuration properties from basyx.*.feature.search.* to basyx.*.feature.experimental.search.*
  • Modifies Java classes to use the new experimental feature flag pattern
  • Updates example configurations and application properties files to reflect the experimental status

Reviewed Changes

Copilot reviewed 36 out of 37 changed files in this pull request and generated no comments.

Show a summary per file
File Description
examples/BaSyxQueryLanguage/basyx/*.yml Updates YAML configuration to nest search under experimental features
examples/BaSyxQueryLanguage/basyx/aas-env.properties Updates property keys to include experimental flag
basyx.*/src/main/resources/application.properties Updates commented configuration examples with experimental flag
basyx.-feature-search/src/test/resources/application. Updates test configurations with experimental property keys
basyx.-feature-search/src/main/java/**/.java Updates Java classes to reference experimental feature flags

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@aaronzi aaronzi merged commit da98fcd into eclipse-basyx:main Aug 22, 2025
43 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants