Skip to content

Conversation

@kendallstrautman
Copy link
Contributor

@kendallstrautman kendallstrautman commented Dec 3, 2025

Description

Bumps version to v4.29.0 and fixes test assertion.

Documentation

Does this require changes to the WorkOS Docs? E.g. the API Reference or code snippets need updates.

[ ] Yes

If yes, link a related docs PR and add a docs maintainer as a reviewer. Their approval is required.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Dec 3, 2025

Greptile Overview

Greptile Summary

Version bump from 4.28.0 to 4.29.0, which includes the recent addition of roles to directory user and SSO profile functionality from PR #311.

  • Simple version constant update in lib/Version.php
  • No code logic changes or potential issues
  • Follows standard versioning practice for minor release

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • Score reflects a simple version constant update with no code logic changes, security implications, or potential for runtime errors
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
lib/Version.php 5/5 Version constant bumped from 4.28.0 to 4.29.0

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant Ver as lib/Version.php
    participant SDK as WorkOS PHP SDK
    
    Dev->>Ver: Update SDK_VERSION constant
    Ver->>Ver: Change "4.28.0" to "4.29.0"
    SDK->>Ver: Read SDK_VERSION for API calls
    Ver-->>SDK: Return "4.29.0"
    SDK->>SDK: Include version in User-Agent header
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@kendallstrautman kendallstrautman merged commit 101d380 into main Dec 3, 2025
7 checks passed
@kendallstrautman kendallstrautman deleted the version-bump-v4.29.0 branch December 3, 2025 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants