Skip to content

Fix errors due to review changes link in review comments#2367

Merged
Urgau merged 2 commits intorust-lang:masterfrom
Urgau:gh-changes-since-fixes
Apr 7, 2026
Merged

Fix errors due to review changes link in review comments#2367
Urgau merged 2 commits intorust-lang:masterfrom
Urgau:gh-changes-since-fixes

Conversation

@Urgau
Copy link
Copy Markdown
Member

@Urgau Urgau commented Apr 7, 2026

Follow-up to #2364 to fixes two errors:

  • normal comments don't have base, head fields, let's not errors on those for normal comments
  • for someone reason GitHub doesn't use the same case between the REST api and webhook, oh well

@Urgau Urgau requested a review from Kobzol April 7, 2026 17:50
@Urgau
Copy link
Copy Markdown
Member Author

Urgau commented Apr 7, 2026

I tested for changes locally, they both fix the issues I'm seeing in the logs.

@Urgau Urgau added this pull request to the merge queue Apr 7, 2026
Merged via the queue into rust-lang:master with commit 271dd30 Apr 7, 2026
3 checks passed
@Urgau Urgau deleted the gh-changes-since-fixes branch April 7, 2026 18:05
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.

2 participants