Skip to content

Conversation

@shuuji3
Copy link
Collaborator

@shuuji3 shuuji3 commented Feb 22, 2025

fix #67

This seems to be working but I only checked related codes in type generate script. There might be something I missed.

related upstream change: bluesky-social/atproto#2378

@netlify
Copy link

netlify bot commented Feb 22, 2025

Deploy Preview for tsky canceled.

Name Link
🔨 Latest commit adf4359
🔍 Latest deploy log https://app.netlify.com/sites/tsky/deploys/67ba1ca47922a000081ff8fa

@github-actions
Copy link
Contributor

github-actions bot commented Feb 22, 2025

Coverage Report for ./packages/client/

Status Category Percentage Covered / Total
🔵 Lines 21.73% 40 / 184
🔵 Statements 21.62% 40 / 185
🔵 Functions 19.6% 20 / 102
🔵 Branches 27.65% 13 / 47
File CoverageNo changed files found.
Generated in workflow #191 for commit adf4359 by the Vitest Coverage Report Action

@shuuji3 shuuji3 force-pushed the shuuji3/fix/lexicons branch from ef3e26f to adf4359 Compare February 22, 2025 18:51
@shuuji3 shuuji3 changed the title chore(lexicons): avoid excluding scripts dir for correct typechecking fix(lexicons): support tid and record-key format in object property Feb 22, 2025
@shuuji3
Copy link
Collaborator Author

shuuji3 commented Feb 23, 2025

Let's merge this to stop CI error. At least, I couldn't find rules that prohibit using minLength with record-key, and the constant (length <= 512) itself is valid one (https://atproto.com/specs/record-key).

@shuuji3 shuuji3 merged commit cb03952 into main Feb 23, 2025
10 checks passed
@shuuji3 shuuji3 deleted the shuuji3/fix/lexicons branch February 23, 2025 09:41
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.

Update lexicon workflow is failing

3 participants