Skip to content

docs: swap group_id & team_slug in import#3184

Merged
stevehipwell merged 1 commit intointegrations:mainfrom
kpocius:main
Feb 12, 2026
Merged

docs: swap group_id & team_slug in import#3184
stevehipwell merged 1 commit intointegrations:mainfrom
kpocius:main

Conversation

@kpocius
Copy link
Contributor

@kpocius kpocius commented Feb 12, 2026

Resolves N/A

Before the change?

Trying to use import as documented:

import {
  to = github_emu_group_mapping.this
  id = "some-team:123456"
}

Would result in a cryptic error:

Error: strconv.Atoi: parsing "some-team": invalid syntax

After the change?

Documentation updated in line with the actual code and the import works as expected:

import {
  to = github_emu_group_mapping.this
  id = "123456:some-team"
}

Pull request checklist

  • Schema migrations have been created if needed (example)
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes
  • No

@github-actions
Copy link

👋 Hi! Thank you for this contribution! Just to let you know, our GitHub SDK team does a round of issue and PR reviews twice a week, every Monday and Friday! We have a process in place for prioritizing and responding to your input. Because you are a part of this community please feel free to comment, add to, or pick up any issues/PRs that are labeled with Status: Up for grabs. You & others like you are the reason all of this works! So thank you & happy coding! 🚀

@deiga
Copy link
Collaborator

deiga commented Feb 12, 2026

Fudge 🤦 I forgot to update the pattern.

Thank you very much for adding this change!

Copy link
Collaborator

@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

@stevehipwell stevehipwell enabled auto-merge (squash) February 12, 2026 20:29
@stevehipwell stevehipwell merged commit 361416c into integrations:main Feb 12, 2026
8 checks passed
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.

3 participants