Skip to content

Conversation

kushudai
Copy link
Contributor

@kushudai kushudai commented Sep 28, 2025

What problem are we solving?

We never released the ACL changes.
Also opentelemetry is out of date.

How are we solving the problem?

Upgrade opentelemetery and release.
Also ugprade rust to latest.

Checks

Please check these off before promoting the pull request to non-draft status.

  • All CI checks are green.
  • I have reviewed the proposed changes myself.

What changes are being made?

  • Upgrades Rust from 1.87 to 1.90 in all GitHub workflows and updates opentelemetry from 0.30 to 0.31.
  • Adds rs-consul crate version 0.13.0 with ACL support and request type builders.
  • Refactors conditional checks for readability and removes unused variable assignments in function calls.

  • AI-assisted summary reviewed and verified by author

@kushudai kushudai marked this pull request as ready for review September 28, 2025 04:49
@kushudai kushudai merged commit 16e9a9b into main Sep 28, 2025
4 checks passed
@kushudai kushudai deleted the upgrade-ot-and-release branch September 28, 2025 04:50
Copy link
Collaborator

@RobloxBuilderAI RobloxBuilderAI left a comment

Choose a reason for hiding this comment

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

✅ BuilderAI analyzed 11 changed files in this PR and found no major concerns.

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