docs(source-github): fix changelog date, add Projects Classic deprecation note, improve incremental sync docs#76075
Closed
devin-ai-integration[bot] wants to merge 1 commit into
Closed
Conversation
…tion note, improve incremental sync docs 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
|
| @@ -141,38 +141,19 @@ This connector outputs the following incremental streams: | |||
| ### Entity-Relationship Diagram (ERD) | |||
| <EntityRelationshipDiagram></EntityRelationshipDiagram> | |||
Contributor
There was a problem hiding this comment.
[markdownlint-fix] reported by reviewdog 🐶
Suggested change
| <EntityRelationshipDiagram></EntityRelationshipDiagram> | |
| <EntityRelationshipDiagram></EntityRelationshipDiagram> |
Contributor
|
Deploy preview for airbyte-docs ready! ✅ Preview Built with commit 27ca97e. |
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_runsandworkflow_jobsboth use SemiIncrementalMixin. Pure incremental streams (comments,commits,issues,review_comments) use IncrementalMixin.Areas of Concern
\*footnote markers on the Projects (Classic) streams — verify these render as literal asterisks in Docusaurus rather than being interpreted as markdown emphasis.What
Documentation improvements for the source-github connector, triggered by review of #76038 (v2.1.16).
Three issues identified via documentation-review-standards review:
/bump-versionran, not when the PR merged.start_dateguidance.How
2026-04-02to2026-04-03to match the actual PR merge date.\*markers to the three Projects (Classic) stream entries with a footnote explaining the upstream deprecation and GHES timeline.start_datewarning into a:::noteadmonition for better visibility.Review guide
docs/integrations/sources/github.md— the only file changed.Items to verify:
\*footnote marker render correctly in Docusaurus? (It should produce a literal asterisk.)assignees,branches,collaborators, etc.). This list was removed here because it already appears in the setup guide (step 7, lines 70–72). Confirm this is acceptable or if the duplication should be preserved.User Impact
Readers of the source-github documentation get:
No negative side effects expected.
Can this PR be safely reverted and rolled back?
Link to Devin session: https://app.devin.ai/sessions/603e417eae8c4448b7a1cfcc7842d04f