fix: enhance translatedVariableDictionary (#17)

leginote / assembly-ts

Korea's national assembly open api mapping

MIT License

1 stars 1 forks source link

Closed sunrabbit123 closed 3 months ago

sunrabbit123 commented 3 months ago

ref: #17

일단 초안 자동완성은 작동합니다.

sunrabbit123 commented 3 months ago

TemplateStringArray는 추론이 안됩니다.

해당 인터페이스를 유지하려면, template에 값이 다 비어있도록 의도해야하며, 이는 의도치않은 버그를 낳을 수 있습니다.

그렇다고 템플릿함수를 제거하고 일반적인 함수 형태를 취하자니, 그는 이쁘지않습니다.

좋은 아이디어 없읍니까?

sunrabbit123 commented 3 months ago

근데 이거 굳이 object 형태여야하나 차피 internal에서만 갖다쓰니 export const 때려도 될 것 같은데

codecov[bot] commented 3 months ago

All modified and coverable lines are covered by tests :white_check_mark:

Files	Coverage Δ
src/bill/getBillList.ts	`100.00% <100.00%> (ø)`
src/constant/translatedVariableDictionary/index.ts	`100.00% <100.00%> (ø)`
.../constant/translatedVariableDictionary/variable.ts	`100.00% <100.00%> (ø)`
src/lawmaker/getLawmakerList.ts	`100.00% <ø> (ø)`
...c/national-assembly/getNationalAssemblySchedule.ts	`100.00% <ø> (ø)`

StatPan commented 3 months ago

제가 미적 감각이 없어서 그런가 이쁜것보다는 기능적인 것에 더 집중하는 경향이..

차피 internal에서만 갖다쓰니 export const 때려도 될 것 같은데

쌉인정인 부분입니다

sunrabbit123 commented 3 months ago

그럼 export const로 전수 변경 해야겠군요 트리쉐이킹을 위해

sunrabbit123 commented 3 months ago

@StatPan 수정완