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

一つの欄に複数の役員情報が記載されているケースに対応 #111

Merged
merged 2 commits into from
Jul 15, 2024

Conversation

yokonao
Copy link
Collaborator

@yokonao yokonao commented Jul 15, 2024

sample770 は一つの役員欄に代表社員と職務執行者の情報が載っている
これらを両方 HoujinExecutiveValues に反映するようにしました

Copy link

Code Metrics Report

main (b1b711e) #111 (3e1a110) +/-
Coverage 6.2% 6.1% -0.1%
Code to Test Ratio 1:0.0 1:0.0 -0.0
Test Execution Time 26s 31s +5s
Details
  |                     | main (b1b711e) | #111 (3e1a110) |  +/-  |
  |---------------------|----------------|----------------|-------|
- | Coverage            |           6.2% |           6.1% | -0.1% |
  |   Files             |             12 |             12 |     0 |
  |   Lines             |            614 |            622 |    +8 |
  |   Covered           |             38 |             38 |     0 |
- | Code to Test Ratio  |          1:0.0 |          1:0.0 |  -0.0 |
  |   Code              |           7876 |           7893 |   +17 |
  |   Test              |            198 |            198 |     0 |
- | Test Execution Time |            26s |            31s |   +5s |

Code coverage of files in pull request scope (0.0% → 0.0%)

Files Coverage +/-
toukibo/parse_body.go 0.0% 0.0%

Reported by octocov

@tychy tychy marked this pull request as ready for review July 15, 2024 07:26
@tychy
Copy link
Owner

tychy commented Jul 15, 2024

LGTM
close #99

@tychy tychy merged commit 6a7f305 into main Jul 15, 2024
2 checks passed
@tychy tychy deleted the fix-99-2 branch July 15, 2024 07:27
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