-
Notifications
You must be signed in to change notification settings - Fork 2
fix(galileo-adk): Rebrand to SPLUNK_AO_* env vars and SplunkAO* classes and rename package to splunk-ao-adk and reset to v0.1.0 #32
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
Merged
Merged
Changes from all commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
375c0be
feat(galileo-adk): rebrand to SPLUNK_AO_* env vars and SplunkAO* clas…
adityamehra 788e932
fix(galileo-adk): rename package to splunk-ao-adk and reset to v0.1.0
adityamehra 97db4ff
Merge branch 'main' into fix/splunk-ao-adk-pyproject
adityamehra 67476e6
refactor(galileo-adk): rename dir to splunk-ao-adk and module to splu…
adityamehra bcdc1a5
fix(splunk-ao-adk): correct version ranges and restore galileo-core[t…
adityamehra 9a60083
Merge branch 'main' into fix/splunk-ao-adk-pyproject
adityamehra 5c7b5ff
ci: add GitHub Actions workflow to test splunk-ao-adk
adityamehra 143c00b
fix(ci): use uv run pip list to avoid VIRTUAL_ENV override in show-ve…
adityamehra b8c1baf
fix(ci): use pip show instead of pip list grep for installed versions…
adityamehra 3f6e4d9
fix(ci): use uv tree --depth 1 for show-versions step to avoid VIRTUA…
adityamehra ca6ce24
fix(dev): pin PyPI as default uv index to bypass Splunk Artifactory i…
adityamehra e369895
fix(dev): add uv.toml to splunk-ao-adk to replace global Artifactory …
adityamehra c9eda22
fix(ci): update python matrix
adityamehra 0adcca2
fix(dev): rename missed galileo logger
adityamehra 653bc4d
fix(ci): Add cache
adityamehra abf7d4e
fix(ci): cache for uv not supported
adityamehra 18c0829
fix(ci): point uv cache-dependency-glob to action's uv.lock for prope…
adityamehra bd8ea35
fix(ci): point uv cache-dependency-glob to action's pyproject.toml
adityamehra 7d50a76
fix(ci): point uv cache-dependency-glob to action's pyproject.toml
adityamehra 15dc3cf
docs(splunk-ao-adk): fix GalileoLogger import and rebrand CONTRIBUTIN…
adityamehra 6189795
Merge branch 'main' into fix/splunk-ao-adk-pyproject
adityamehra 58ac9c4
fix(docs): Use galileo docs
adityamehra 28f65b2
test(config): add unit tests for _bridge_env_vars() including SPLUNK_…
adityamehra 84e59ad
fix(config): remove SPLUNK_AO_API_URL bridge entry and fix test isola…
adityamehra 6841cd0
fix(docs): Update to python 3.11+
adityamehra File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,62 @@ | ||
| name: Test splunk-ao-adk | ||
|
|
||
| on: | ||
| push: | ||
| branches: [main] | ||
| paths: | ||
| - 'splunk-ao-adk/**' | ||
| - '.github/workflows/ci-tests-splunk-ao-adk.yaml' | ||
| pull_request: | ||
| paths: | ||
| - 'splunk-ao-adk/**' | ||
| - '.github/workflows/ci-tests-splunk-ao-adk.yaml' | ||
|
|
||
| concurrency: | ||
| group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }} | ||
| cancel-in-progress: true | ||
|
|
||
| jobs: | ||
| test: | ||
| name: Python ${{ matrix.python-version }} | ||
| strategy: | ||
| fail-fast: false | ||
| matrix: | ||
| python-version: ["3.11", "3.12", "3.13", "3.14"] | ||
|
|
||
| runs-on: ubuntu-latest | ||
| # Hard cap per matrix job to bail out fast on real hangs. | ||
| timeout-minutes: 30 | ||
|
|
||
| defaults: | ||
| run: | ||
| working-directory: splunk-ao-adk | ||
|
|
||
| steps: | ||
| - name: Checkout | ||
| uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2 | ||
|
|
||
| - name: Set up Python ${{ matrix.python-version }} | ||
| uses: actions/setup-python@a309ff8b426b58ec0e2a45f0f869d46889d02405 # v6 | ||
| with: | ||
| python-version: ${{ matrix.python-version }} | ||
|
|
||
| - name: Install uv | ||
| uses: astral-sh/setup-uv@681c641aba71e4a1c380be3ab5e12ad51f415867 # v7.1.6 | ||
| with: | ||
| enable-cache: true | ||
| cache-dependency-glob: "**/splunk-ao-adk/pyproject.toml" | ||
|
|
||
| # uv sources in pyproject.toml pins splunk-ao to the local monorepo root | ||
| # as an editable install, so this always tests against the latest local code. | ||
| # pytest pythonpath includes ".." (the repo root) so test_support is importable. | ||
| - name: Install dependencies | ||
| run: uv sync --dev | ||
|
|
||
| - name: Show installed versions | ||
| run: uv pip list | grep -E "^(galileo|google-adk|splunk-ao)" | ||
|
|
||
| - name: Run type check | ||
| run: uv run mypy src/ | ||
|
|
||
| - name: Run tests | ||
| run: uv run pytest --cov=splunk_ao_adk --cov-report=xml |
This file was deleted.
Oops, something went wrong.
File renamed without changes.
File renamed without changes.
This file contains hidden or 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
Oops, something went wrong.
Oops, something went wrong.
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.
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.
🟡 minor (documentation): This file was only partially rebranded — the diff changed just two lines (
src/galileo_adk/path and the--covflag). The rest still reads as the old package: the heading# Contributing to galileo-adk, links torungalileo/galileo-python, the architecture diagram class names (GalileoADKPlugin,GalileoADKCallback,GalileoObserver,GalileoLogger,GalileoBaseHandler),cd galileo-adk, thegalileo>=1.45.0dependency example, theRelease galileo-adkworkflow reference, anddocs.rungalileo.io. Since rebranding the docs is an explicit goal of this PR, this file should be updated for consistency with README.md and the renamed source.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.
Change what is reasonable, but it is OK to leave some of this documentation to be updated at a later time.