Skip to content
This repository was archived by the owner on Apr 15, 2026. It is now read-only.

refactor(scim): move enterpriseSCIMMetaSchema() to util_enterprise_scim.go#61

Merged
vmvarela merged 1 commit intoenterprise-scimfrom
fix/54-scim-meta-schema-util
Mar 23, 2026
Merged

refactor(scim): move enterpriseSCIMMetaSchema() to util_enterprise_scim.go#61
vmvarela merged 1 commit intoenterprise-scimfrom
fix/54-scim-meta-schema-util

Conversation

@vmvarela
Copy link
Copy Markdown
Owner

Summary

Closes #54

Pure refactor — no functional change.

enterpriseSCIMMetaSchema() was defined in data_source_github_enterprise_scim_groups.go but referenced by both the groups and users data sources. Moving it to util_enterprise_scim.go (which already houses shared pagination and flatten utilities for this feature) makes the codebase easier to navigate.

Changes

File Change
github/util_enterprise_scim.go Added enterpriseSCIMMetaSchema() + schema import
github/data_source_github_enterprise_scim_groups.go Removed enterpriseSCIMMetaSchema()

Verification

go build ./github/... passes cleanly.

…im.go

Pure refactor — no functional change. The shared schema helper was
defined in data_source_github_enterprise_scim_groups.go but referenced
by both groups and users data sources. Moving it to the shared util
file makes the codebase easier to navigate.

Closes #54
@vmvarela vmvarela added the type:chore Maintenance, refactoring, tooling label Mar 23, 2026
@github-actions github-actions Bot added type:feature New functionality type:bug Something isn't working labels Mar 23, 2026
@vmvarela vmvarela merged commit 60ba689 into enterprise-scim Mar 23, 2026
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

type:bug Something isn't working type:chore Maintenance, refactoring, tooling type:feature New functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant