Skip to content

BlueskyをAccountTypeに追加#745

Merged
Tennessine699 merged 6 commits into
mainfrom
feat/add_account_bluesky
Jul 1, 2025
Merged

BlueskyをAccountTypeに追加#745
Tennessine699 merged 6 commits into
mainfrom
feat/add_account_bluesky

Conversation

@Tennessine699

Copy link
Copy Markdown
Contributor

resolves #738

Blueskyのプロフィールリンクの正規表現が見つからなかったのでいろいろ試してこれに落ち着きました
多分大丈夫だと思います

@codecov

codecov Bot commented Jan 12, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.

Project coverage is 70.66%. Comparing base (ab347f8) to head (b015dff).
Report is 7 commits behind head on main.

Files with missing lines Patch % Lines
internal/domain/user.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #745      +/-   ##
==========================================
- Coverage   70.77%   70.66%   -0.12%     
==========================================
  Files          43       43              
  Lines        4572     4578       +6     
==========================================
- Hits         3236     3235       -1     
- Misses       1183     1188       +5     
- Partials      153      155       +2     
Flag Coverage Δ
integration 70.66% <83.33%> (-0.12%) ⬇️
unit 70.66% <83.33%> (-0.12%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread internal/pkgs/random/random.go Outdated
Comment on lines -407 to +408
httpError(t, "Bad Request: validate error: type: must be no greater than 11."),
httpError(t, fmt.Sprintf("Bad Request: validate error: type: must be no greater than %d.", domain.AccountLimit-1)),

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ハードコードしてたのか

Comment on lines -407 to +408
httpError(t, "Bad Request: validate error: type: must be no greater than 11."),
httpError(t, fmt.Sprintf("Bad Request: validate error: type: must be no greater than %d.", domain.AccountLimit-1)),

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ハードコードしてたのか

@Pugma Pugma left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

公式情報もなさそうですし、とりあえずはこちらの正規表現で問題ないかと思います
もし不具合があればその際に対応する形で問題ないかと思います

@Tennessine699
Tennessine699 merged commit 6afe821 into main Jul 1, 2025
14 checks passed
@Tennessine699
Tennessine699 deleted the feat/add_account_bluesky branch July 1, 2025 13:11
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.

bluesky

3 participants