Skip to content

Todo #924

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.

Sign up for GitHub

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

Jump to bottom
Open
th30neee wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Todo #924

th30neee wants to merge 9 commits into from

Conversation

@th30neee
Copy link

@th30neee th30neee commented Dec 10, 2025 •
edited
Loading

Pre-review checklist:

  • I have run a spell check on the write-up and made sure no typos exist.
  • References to Snapshot/governance forum are correct on the AIP. If no snapshot exists, make sure no TODO's exist.
  • The specification on the writeup is aligned with the forum, snapshot, and the payload contract. If there are any changes, they are explicitly mentioned/communicated.
  • Minimal tests exist, and the snapshot diff report generated is the latest one and aligned with the payload.
  • If deploy scripts are manually updated from the generated ones, I have carefully validated that they are correct, including the deploy commands and the proposal-creation script.
  • If the aave-helpers submodule is updated, I have validated it is pointing to the latest version.
  • I have validated that no unused files and imports are being added.
  • For an asset listing, the write-up includes a detailed specification of the price feed used, CAPO adapters (with each CAPO layer described separately), and eModes (with tables) if changed.
  • Wherever possible, I have validated that addresses from the address book are used instead of raw addresses.

dependabot bot and others added 9 commits November 18, 2025 10:28
@dependabot
chore(deps): bump js-yaml from 3.14.1 to 3.14.2
d34a1bc 
Bumps [js-yaml](https://github.com/nodeca/js-yaml) from 3.14.1 to 3.14.2.
- [Changelog](https://github.com/nodeca/js-yaml/blob/master/CHANGELOG.md)
- [Commits](nodeca/js-yaml@3.14.1...3.14.2)

---
updated-dependencies:
- dependency-name: js-yaml
  dependency-version: 3.14.2
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
@th30neee
Merge pull request #2 from th30neee/dependabot/npm_and_yarn/js-yaml-3…
84c98d0 
….14.2
feat: add OpenTelemetry instrumentation for Kubiks observability
886b0df 
- Install OpenTelemetry SDK and dependencies
- Create instrumentation.ts with NodeSDK configuration
- Support for OTEL_EXPORTER_OTLP_ENDPOINT and OTEL_SERVICE_NAME env vars
- Automatically instrument all generator commands with tracing
- Add Kubiks configuration to .env.example
fix: improve OpenTelemetry instrumentation compatibility
53bd738 
- Use CommonJS requires for better compatibility with tsx
- Parse headers correctly from OTEL_EXPORTER_OTLP_HEADERS environment variable
- Improve error handling and logging
- Remove type annotations that cause issues with require()
@th30neee
Merge pull request #3 from th30neee/feature/add-opentelemetry-instrum…
b80f930 
…entation
@th30neee
Merge branch 'main' into Munyun
c0c345f
@th30neee
Merge branch 'bgd-labs:main' into Munyun
2f56e70
@th30neee
Merge branch 'bgd-labs:main' into Munyun
ce0a328
@th30neee
Merge branch 'bgd-labs:main' into Munyun
17e33a1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@th30neee