Skip to content
This repository was archived by the owner on Sep 11, 2025. It is now read-only.

Conversation

@mattjohnsonpint
Copy link
Contributor

  • Use Go 1.24 to build the Modus Runtime
  • Update tests to use the new text context provided by Go 1.24
  • Update trunk plugins
  • Update a few runtime dependencies to resolve security warning
  • Update API explorer dependencies
image

@mattjohnsonpint mattjohnsonpint requested review from a team and Copilot March 7, 2025 20:35
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

PR Overview

This PR makes miscellaneous updates including switching to Go 1.24 for builds, updating test contexts, and bumping various dependency versions for security and feature updates.

  • Updated dependencies and versions in trunk configuration
  • Replaced context.Background() with t.Context() in multiple test files
  • Updated integration test context management for PostgreSQL container setup

Reviewed Changes

File Description
.trunk/trunk.yaml Dependency version updates for build and security improvements
runtime/utils/http_test.go Updated to use t.Context() for improved context handling in tests
runtime/integration_tests/postgresql_integration_test.go Updated context handling in container setup and teardown
runtime/graphql/schemagen/schemagen_go_test.go Replaced context.Background() with t.Context() in tests
runtime/graphql/schemagen/schemagen_as_test.go Adjusted context usage in assembly script schema tests
runtime/collections/in_mem/text_index_test.go Changed context initialization to use t.Context() in tests
runtime/collections/in_mem/sequential/vector_index_test.go Updated context usage in sequential tests
runtime/collections/in_mem/hnsw/vector_index_test.go Modified context acquisition from Background() to t.Context()
runtime/models/models_test.go Updated models test to use t.Context() for consistency

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

@mattjohnsonpint mattjohnsonpint merged commit 2442ed1 into main Mar 7, 2025
54 checks passed
@mattjohnsonpint mattjohnsonpint deleted the mjp/misc branch March 7, 2025 20:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants