bezier-react icon indicating copy to clipboard operation
bezier-react copied to clipboard

Enhance `validate-token` plugin of `stylelint-bezier`

Open yangwooseong opened this issue 1 year ago • 1 comments

Description

validate-token 플러그인을 개선합니다.

Reasons for suggestion

현재 css variable에 기본값을 넣으면 린트에 걸리고 있습니다. bezier-token 을 사용하는지 아닌지를 검사하는 것이 목적이기 때문에 기본값 사용을 허용해야 합니다.

Proposed solution

  1. css variable 안에 기본값을 포함하는 것을 허용합니다. 지금은 아래처럼 기본값을 넣으면 린트에 걸립니다.
image
  1. 에러 메시지에 ignorePrefix에 대한 정보를 추가합니다. 위 예의 경우는 Invalid token: "token-name". Use tokens from @channel.io/bezier-tokens, or add some prefix to "ignorePrefix" of plugin option. (bezier/validate-token)Stylelint(bezier/validate-token) 정도라면 좋을 것 같습니다.

  2. 테스트 코드를 추가해서 체크가 통과하는 경우와 실패하는 경우를 명확히 알게 합니다. bezier-token을 import 하고 있지만 validate(value, option, tokens) 같은 함수를 분리한다면 충분히 테스트 코드 작성이 가능할 것 같습니다.

References

No response

yangwooseong avatar Oct 29 '24 08:10 yangwooseong

https://desk.channel.io/#/channels/1/team_chats/groups/1237/672097a2d131bd94a195

channeltalk[bot] avatar Oct 29 '24 08:10 channeltalk[bot]