-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: remove redundant content from connector's READMEs (discussing, do not merge) #52698
base: master
Are you sure you want to change the base?
Conversation
…freshchat READMEs - Remove duplicated development and publishing instructions - Add links to authoritative documentation sources - Keep connector-specific information intact - Maintain consistent documentation structure Co-Authored-By: Aaron <AJ> Steers <[email protected]>
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
🤖 Devin AI EngineerOriginal prompt from Aaron:
I'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
⚙️ Control Options:
|
Co-Authored-By: Aaron <AJ> Steers <[email protected]>
…rypt READMEs Co-Authored-By: Aaron <AJ> Steers <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the consolidation here and the consistency from one readme to the next. I'll assume we've already verified the it was indeed correct to remove the existing content. If that's the case, this looks good.
@ian-at-airbyte - thanks for the review! I can't say if all the removed content is stale or not but I can look into this further. And to be clear, there's no rush to merge this; I'd just like to work towards a place where there's a single place to update connector maintenance and "contributing" guides for each language / type. |
Co-Authored-By: Aaron <AJ> Steers <[email protected]>
Co-Authored-By: Aaron <AJ> Steers <[email protected]>
Got it. That makes sense. I do like the AI content participation as a whole but to borrow the Russian proverb, we might want to adopt a "Trust, but verify" philosophy, at least for now. |
This PR removes redundant content from connector-level READMEs and adds links to authoritative documentation.
Changes:
Link to Devin run: https://app.devin.ai/sessions/61a22ee817e44978b6323f4963f54bf5