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

[Discussion] Remove current keys from checks.json #67

Open
michael-proulx opened this issue Nov 13, 2024 · 2 comments
Open

[Discussion] Remove current keys from checks.json #67

michael-proulx opened this issue Nov 13, 2024 · 2 comments

Comments

@michael-proulx
Copy link

Would like to know what will be the impact to remove keys that are updated(current) from the checks.json.

After splitting our markdown file into smaller chunk we do have many keys.

"current": false,
"baseTotal": 30444,
"total": 32006,
"states": {
  "current": 27186,
  "outdated": 72,
  "missing": 3186,
  "unused": 4748
},
"keys": {
  ...all 32006 keys
  Would be easier to only see outdated, missing and unused?
}

So there is many keys that do not need our attention in the checks.json

What your thought on this @bglw?

@bglw
Copy link
Contributor

bglw commented Nov 15, 2024

To clarify, would this be addressed by adding a flag to the checks command? Something like --omit-current to avoid writing the current keys into the file? I can add that if so.

@michael-proulx
Copy link
Author

That will be a good option without compromising the people who want them.

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

No branches or pull requests

2 participants