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

add a mask and WHERE part #158

Closed
ykostsik opened this issue Feb 3, 2020 · 4 comments · Fixed by #160
Closed

add a mask and WHERE part #158

ykostsik opened this issue Feb 3, 2020 · 4 comments · Fixed by #160
Labels

Comments

@ykostsik
Copy link

ykostsik commented Feb 3, 2020

Feature request
Dear all,
Is it possible to add( bc I didn't find a such option) an option to mask the table or a column name using * , %, _, etc? Or check some conditions likewhere: to identify the list of affected objects when the exclude list is specified.
Thanks in advance.

@ykostsik ykostsik added the enhancement New feature or request label Feb 3, 2020
@ykostsik ykostsik changed the title add conditions add a mask and WHERE part Feb 3, 2020
@k1LoW
Copy link
Owner

k1LoW commented Feb 4, 2020

Hi @ykostsik !

Why do you want to mask the table/column name ?

@ykostsik
Copy link
Author

ykostsik commented Feb 4, 2020

Hi @k1LoW,
thanks for quick response.
For instance, I want to ensure that every column has comments, except the column with the primary key, that is named by pattern for every table in the database - %_id, so it would be enough to add - "*id" to exclude such columns from the whole database.

@k1LoW
Copy link
Owner

k1LoW commented Feb 4, 2020

@ykostsik

For instance, I want to ensure that every column has comments, except the column with the primary key, that is named by pattern for every table in the database

Are the settings you want to do like below? ( see tbls lint section )

lint:
  requireTableComment:
    enabled: true
  requireColumnComment:
    enabled: true
    exclude:
      - id
      - *_id  # 👈your request

@ykostsik
Copy link
Author

ykostsik commented Feb 4, 2020

@k1LoW yes, something like this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants