Skip to content

collab: Only access github_user_id and github_login columns in tests - #58457

Merged
maxdeviant merged 2 commits into
mainfrom
push-ynpnqvumluvo
Jun 3, 2026
Merged

collab: Only access github_user_id and github_login columns in tests#58457
maxdeviant merged 2 commits into
mainfrom
push-ynpnqvumluvo

Conversation

@maxdeviant

Copy link
Copy Markdown
Member

This PR marks the github_user_id and github_login columns on the User database model as only being accessible in tests.

Closes CLO-829.

Release Notes:

  • N/A

@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Jun 3, 2026
@maxdeviant
maxdeviant requested a review from tomhoule June 3, 2026 18:37
@zed-community-bot zed-community-bot Bot added the staff Pull requests authored by a current member of Zed staff label Jun 3, 2026
@tomhoule

tomhoule commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Hm, tests are running into that "Unknown".

@maxdeviant

Copy link
Copy Markdown
Member Author

Hm, tests are running into that "Unknown".

I needed to update the test assertion.

@maxdeviant
maxdeviant added this pull request to the merge queue Jun 3, 2026
Merged via the queue into main with commit c4cb8ae Jun 3, 2026
32 checks passed
@maxdeviant
maxdeviant deleted the push-ynpnqvumluvo branch June 3, 2026 19:23
TomPlanche pushed a commit to TomPlanche/zed that referenced this pull request Jun 8, 2026
…sts (zed-industries#58457)

This PR marks the `github_user_id` and `github_login` columns on the
`User` database model as only being accessible in tests.

Closes CLO-829.

Release Notes:

- N/A
This was referenced Jun 18, 2026
jonx pushed a commit to jonx/zed-aros that referenced this pull request Jul 17, 2026
…sts (zed-industries#58457)

This PR marks the `github_user_id` and `github_login` columns on the
`User` database model as only being accessible in tests.

Closes CLO-829.

Release Notes:

- N/A
jolutz pushed a commit to jolutz/zed that referenced this pull request Aug 8, 2026
…sts (zed-industries#58457)

This PR marks the `github_user_id` and `github_login` columns on the
`User` database model as only being accessible in tests.

Closes CLO-829.

Release Notes:

- N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants