Skip to content

Conversation

@wesleyboar
Copy link
Collaborator

@wesleyboar wesleyboar commented Jan 5, 2026

What Did You Change?

Refactored the requirements.txt sync and validation to avoid false negatives.

Do You Want Support (syntax, design, etc)?

No, I think I'm good. AI wrote it. I directed, reviewed, and tested:

  1. Dev changes poetry.lock. Bot edits requirements.txt.
  2. Bot changes requirements.txt from target/base branch content. Bot allows.
  3. Dev changes requirements.txt. Bot prevents.
  4. Dev reverts requirements.txt to target/base branch content. Bot allows.

Any Reference Material Worth Sharing?

Tested on #259.

@wesleyboar wesleyboar requested a review from a team January 5, 2026 21:01
@netlify
Copy link

netlify bot commented Jan 5, 2026

Deploy Preview for ds-user-guide ready!

Name Link
🔨 Latest commit 908938b
🔍 Latest deploy log https://app.netlify.com/projects/ds-user-guide/deploys/695c42d7bc24630008435340
😎 Deploy Preview https://deploy-preview-261--ds-user-guide.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

wesleyboar and others added 23 commits January 5, 2026 15:07
will it update requirements.txt?
will this change be caught by bot?
…m:DesignSafe-CI/DS-User-Guide into refactor/requirements-sync-and-validation
will this change be caught by bot?

previous test got marred by auto-merge because i forgot to pulll beofre push
…m:DesignSafe-CI/DS-User-Guide into refactor/requirements-sync-and-validation
will this change be caught by bot?

testing now since latest refactor commit
will this change be caught by bot?

testing now since latest fix commit
@wesleyboar wesleyboar merged commit c5df7ee into main Jan 5, 2026
@wesleyboar wesleyboar deleted the refactor/requirements-sync-and-validation branch January 5, 2026 23:15
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