Skip to content

Conversation

@elminster-aom
Copy link
Contributor

@elminster-aom elminster-aom commented Dec 4, 2025

BREAKING CHANGE: SCIMEnterpriseDisplayReference.Ref is now of type *string.

Related with #3813

From REST API endpoints for Enterprise SCIM docs, implements:

Breaking API change

When provisioning Groups, the SCIMEnterpriseDisplayReference.Ref is optional for provisioning actions so it changed to pointer.

@elminster-aom elminster-aom mentioned this pull request Dec 4, 2025
12 tasks
@codecov
Copy link

codecov bot commented Dec 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.41%. Comparing base (f093aaa) to head (faf2739).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3852      +/-   ##
==========================================
+ Coverage   92.40%   92.41%   +0.01%     
==========================================
  Files         200      200              
  Lines       14410    14432      +22     
==========================================
+ Hits        13315    13337      +22     
  Misses        895      895              
  Partials      200      200              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@elminster-aom elminster-aom marked this pull request as ready for review December 4, 2025 13:09
@gmlewis gmlewis added the NeedsReview PR is awaiting a review before merging. label Dec 4, 2025
@gmlewis gmlewis changed the title Implement Enterprise SCIM - Provision Groups & Users feat!: Implement Enterprise SCIM - Provision Groups & Users Dec 4, 2025
@gmlewis gmlewis added the Breaking API Change PR will require a bump to the major version num in next release. Look here to see the change(s). label Dec 4, 2025
Copy link
Collaborator

@gmlewis gmlewis left a comment

Choose a reason for hiding this comment

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

Thank you, @elminster-aom!
LGTM.
Awaiting second LGTM+Approval from any other contributor to this repo before merging.

cc: @stevehipwell - @alexandear - @zyfy29

Fix typo on the `CIMEnterpriseGroupAttributes` struct documentation.

Co-authored-by: Steve Hipwell <steve.hipwell@gmail.com>
@elminster-aom
Copy link
Contributor Author

@gmlewis Do I need to take action regarding this drop of -0.10% for codecov/project?

@gmlewis
Copy link
Collaborator

gmlewis commented Dec 9, 2025

@gmlewis Do I need to take action regarding this drop of -0.10% for codecov/project?

No, thank you though, @elminster-aom. The Codecov reports are mostly informational, and I'm not always convinced that they are helpful. Sometimes it will catch cases where unit tests could have added a testBadOptions call, for example, but in this case and many other cases they are simply a bit noisy.

Copy link
Contributor

@stevehipwell stevehipwell left a comment

Choose a reason for hiding this comment

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

LGTM

@gmlewis gmlewis removed the NeedsReview PR is awaiting a review before merging. label Dec 9, 2025
@gmlewis
Copy link
Collaborator

gmlewis commented Dec 9, 2025

@elminster-aom - it looks like some conflicts have emerged from other merges. Would you like to address them, or would you like me to do that? I'm fine either way. Some people like a fun challenge, others don't. 😂 Your call.

* forkstream/master:
  chore!: Remove `PullRequestRuleParameters.AutomaticCopilotCodeReviewEnabled` field (google#3866)
  feat: Add support for `copilot_code_review` rule type (google#3857)
  feat: Implement Enterprise SCIM - Delete Groups or Users (google#3856)
  feat: Add `advanced_search` parameter to `SearchOptions` (google#3868)
  fix!: Change Org usage report `Quantity` to `float64` (google#3862)
  build(deps): Bump actions/checkout from 6.0.0 to 6.0.1 in the actions group (google#3863)
  build(deps): Bump golang.org/x/sync from 0.18.0 to 0.19.0 in /tools (google#3864)
  build(deps): Bump golang.org/x/term from 0.37.0 to 0.38.0 in /example (google#3865)
  feat: Implement Enterprise SCIM - Set Groups or Users (google#3858)
  feat: Add support for Enterprise Team APIs (google#3861)
  Bump go-github from v79 to v80 in /scrape (google#3855)
  Bump version of go-github to v80.0.0 (google#3854)
  feat: Add repository target to ruleset (google#3850)
@elminster-aom
Copy link
Contributor Author

elminster-aom commented Dec 10, 2025

it looks like some conflicts have emerged from other merges. Would you like to address them, ... ?

Sure. Always the fanny option for me, of course 😁 . Thanks @gmlewis

@gmlewis
Copy link
Collaborator

gmlewis commented Dec 10, 2025

Thank you, @stevehipwell and @elminster-aom!
Merging.

@gmlewis gmlewis merged commit 4d9bd08 into google:master Dec 10, 2025
7 checks passed
@elminster-aom elminster-aom deleted the postEnterpriseSCIM branch December 11, 2025 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Breaking API Change PR will require a bump to the major version num in next release. Look here to see the change(s).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants