[publint] skip 옵션 제공

[byhhh2]

Related Issue

• [publint] 사용성 개선 #25

Describe your changes

• 필드별로 verify 분리
  • 필드별로 skip 가능 (이렇게 갈지 고민이 되네요..)
• 오류 메시지 영어로 변환 (gpt)
• verifyPackageJSON에서 오류 메시지 출력 제거
  • 오류 throw만 하고 cli에서 log함

Request

[2-one-week]

@byhhh2 님

필드별로 skip 가능 (이렇게 갈지 고민이 되네요..)

고민인 이유는 필드가 너무 많아져서 그런걸까요?

[byhhh2]

@2-one-week 님
loose/strict 기준이 명확하지가 않아서 필드별로 option을 제공해보았는데요
너무 짜잘한 것 같기도 해서요 🙄

[2-one-week]

loose/strict 기준이 명확하지가 않아서 필드별로 option을 제공해보았는데요

아예 다 스킵할거냐,, 다 검사할거냐 이차이군요ㅋㅋ

저는 개인적으로 자유도가 높은걸 좋아하긴 하는데, 또 사용하는 쪽 생각해보면 단순한게 좋겠다 싶어서,,,

일단 loose / strict 로 가고 니즈가 있으면 세분화하는건 어떠신가용?

---

@yceffort-naver @keemhyunseok 님

이거 의견 부탁드려도 될까요?

[keemhyunseok]

loose / strict 로 가고 니즈가 있으면 세분화하는건 어떠신가용?

저도 간결한게 좋을 듯 하여 이 의견에 동의합니당

[byhhh2]

@2-one-week 님, @keemhyunseok 님
#25 에서 1번 배럴파일 없는 케이스 대응 (main, module, types)
이거는 배럴 파일 없음 옵션으로 추가하는거 어떤가요?

[2-one-week]

이거는 배럴 파일 없음 옵션으로 추가하는거 어떤가요?

필드 하나로 합치잔 말씀이시죠??

[byhhh2]

@2-one-week 님
앗; 제가 중간을 너무 짤라 먹었네요
배럴파일 없는 케이스 대응 (main, module, types)는 loose / strict으로 나누기가 애매할 것 같은데
loose / strict으로 나눌지 배럴 파일 없음 이런 추상화된 옵션을 추가하는게 나을지 여쭤보려 했어요!

[2-one-week]

@byhhh2 님

음 저는 단순하게

try {
  // 여러 검증

} catch (error) {
    if(verify === 'loose'){
          console.warn(error)
    } else if (verify === 'strict') {
       throw error
   } else if (verify === 'slient') {
            // ㅎㅅㅎ
   }
}
 

이렇게 생각했는데요.

패키지 개발이라는게 사실 개발자의 입맛에 따라서 다양한 부분이 다를 수 있어서,

에러를 터트릴거야 말꺼야? 가 주된 문제? 라고 생각합니다.

[byhhh2]

@2-one-week 님
에러 레벨을 조정하는 옵션을 말씀 주신 거였군요!!
좋은 아이디어인 것 같습니다..!!

말씀주신 에러 레벨 옵션을 publint 아닌 pite에만 줘도 될 것 같기는 한데 오떠신가요?
publint를 cli가 아니라 js api로 쓸 때에는 logging을 따로 안하구 error를 던진다거나 error msg를 응답으로 주는 형태로 가려해서용

publint에서 error throw or error msg return => pite에서 error level option을 받아서 어떻게 처리할지 정함

@yceffort-naver 님 @keemhyunseok 님 의견도 부탁드립니당!

[2-one-week]

@byhhh2 님

말씀주신 에러 레벨 옵션을 publint 아닌 pite에만 줘도 될 것 같기는 한데 오떠신가요?

네네 이것도 좋을 것 같습니다!!! pite에서 처리하는게 맞긴 하겠네요