-
Notifications
You must be signed in to change notification settings - Fork 253
Docs monitor changes #1752
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 monitor changes #1752
Conversation
Amp-Thread-ID: https://ampcode.com/threads/T-a46d7a5f-9f65-4892-8708-4bf8b754f84f Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-e3ffb9bc-2bd9-4fb0-afbe-8737bfbfec18 Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-97be7b71-82d3-4928-8274-558ed181c84e Co-authored-by: Amp <amp@ampcode.com>
|
@kenny-stacks what is this failing check: |
|
changes seem fine, just need to address the failing status check |
|
This check is based on a workflow I had in here where Claude did some automated PR reviews but I disabled it, shouldn't run from now on. It doesn't actually affect anything with the PR itself. |
eric-stacks
left a comment
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.
lgtm
Changes
Schedule: Changed from every 6 hours to daily at midnight UTC
Lookback window: Updated default hours_back from 6 to 24 hours
Target repo: Issues are now created in stacks-network/docs (previously pointed to fork for testing)
How it works
Runs daily and fetches merged PRs from monitored repos (stacks-core, clarinet, stacks.js, etc.)
Claude analyzes each PR's diff against existing documentation
Creates GitHub issues for needed doc updates, additions, or deprecations