docs(source-github): fix changelog dates and improve documentation clarity#76128
Closed
devin-ai-integration[bot] wants to merge 2 commits into
Closed
docs(source-github): fix changelog dates and improve documentation clarity#76128devin-ai-integration[bot] wants to merge 2 commits into
devin-ai-integration[bot] wants to merge 2 commits into
Conversation
…arity Co-Authored-By: bot_apk <apk@cognition.ai>
Contributor
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
Contributor
👋 Greetings, Airbyte Team Member!Here are some helpful tips and reminders for your convenience. 💡 Show Tips and TricksPR Slash CommandsAirbyte Maintainers (that's you!) can execute the following slash commands on your PR:
📚 Show Repo GuidanceHelpful Resources
|
…032) Co-Authored-By: bot_apk <apk@cognition.ai>
Contributor
|
Deploy preview for airbyte-docs ready! ✅ Preview Built with commit 8a1347d. |
Contributor
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Documentation Confidence Assessment
Overall Confidence: 3/5
What I Verified vs. What I Inferred
workflow_jobsinherits fromWorkflowRunJobswhich usesSemiIncrementalMixin, confirming it respects Start Date filtering. Changelog dates verified againstgit logmerge commit timestamps for PRs fix(source-github): fix remaining NameError references to removed MessageRepresentationAirbyteTracedErrors #76080 and fix(source-github): replace deprecated MessageRepresentationAirbyteTracedErrors with AirbyteTracedException #76038.Areas of Concern
workflow_jobscorrectly belongs in the Start Date list — it was the only incremental stream omitted from that list.What
Updates the source-github connector documentation triggered by commit 828cdd9 (#76080). Fixes incorrect changelog dates, adds a missing stream to the Start Date list, and improves readability of the Notes and Rate Limiting sections.
How
Changelog date corrections — v2.1.17 date corrected from
2026-04-03to2026-04-07(actual merge date of PR fix(source-github): fix remaining NameError references to removed MessageRepresentationAirbyteTracedErrors #76080); v2.1.16 date corrected from2026-04-02to2026-04-03(actual merge date of PR fix(source-github): replace deprecated MessageRepresentationAirbyteTracedErrors with AirbyteTracedException #76038). Dates were set by/bump-versionat PR creation time, not at merge time.Missing stream in Start Date list — Added
workflow_jobsto the list of streams that respect the Start Date configuration. This stream is incremental and was already documented in the incremental streams section but was omitted from the Start Date applicability list.Notes section cleanup — Removed unnecessary backslash escapes around parentheses, improved grammar and sentence structure, replaced bare
[docs]link text with descriptive link text, and condensed note checkin catalogs #3 for clarity.Rate limiting section — Fixed misplaced
:::tipclosing tag (was indented inside the ordered list, causing a rendering issue), tightened wording.Minor style fixes — Active voice, consistent list capitalization, removed first-person "our", added missing trailing period.
Review guide
docs/integrations/sources/github.md— Single file changed.Human review checklist
workflow_jobscorrectly supports Start Date filtering (it inherits fromSemiIncrementalMixinviaWorkflowRuns)User Impact
More accurate changelog dates and a corrected Start Date stream list. No functional impact — documentation only.
Can this PR be safely reverted and rolled back?
Link to Devin session: https://app.devin.ai/sessions/3ddcb87099f44a2198d7708ebe7fce27