-
Notifications
You must be signed in to change notification settings - Fork 299
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
Clean up lexicons #41
Comments
Is there any reason to keep a lexicons folder instead of using https://www.npmjs.com/package/@atproto/lexicon? I might just be really bad at TypeScript but whenever I try to access an object like |
aendra-rininsland
pushed a commit
to aendra-rininsland/feed-generator
that referenced
this issue
Mar 20, 2024
* Removes src/lexicons * Replaces with imports from respective upstream packages * Replaces ts-node with tsx, namely because @atproto/bsky doesn't export lexicons * See: bluesky-social/atproto#2353 * Merging bluesky-social/atproto#2354 will allow reverting to ts-node Fixes bluesky-social#41.
Open
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I did a dumb copy/paste job in the lexicon folder. clean that up to only necessary lexicons
The text was updated successfully, but these errors were encountered: