Skip to content
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

update & simplify ts-bot/ #16

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

cowtoolz
Copy link

@cowtoolz cowtoolz commented Dec 6, 2024

Code changes:

  • Inline comments
  • Error handling
  • Simplified execution path
  • Updated to modern @atproto/api

Changes:

  • Removed external dependencies
  • Updated remaining dependencies
  • Added npm run scripts for setup/execution
  • Clarified & updated readme to reflect changes

The previous example seemed too intimidating for noobs, which are often drawn to building projects like this but aren't familiar with cron/env/etc. This doesn't compromise any of the value of the example for experienced devs, but makes things (at least I feel) easier to reason about for non-programmers by adding basic tooling and simplifying the code.

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.

1 participant