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

fix(ci): Remove fix-flaws command from auto-cleanup-bot.yml #38332

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

OnkarRuikar
Copy link
Contributor

The yarn content fix-flaws has stopped working. Let's remove it till it is implemented in rari.

$ yarn content fix-flaws

yarn run v1.22.22
$ yarn -s info:rari && env-cmd --silent cross-env CONTENT_ROOT=files rari-tool fix-flaws

🐥 This command is now using rari: https://github.com/mdn/rari
🐞 Please report any issues here:  https://github.com/mdn/rari/issues/new?template=bug.yml

error: unrecognized subcommand 'fix-flaws'

Usage: rari content [OPTIONS] <COMMAND>

For more information, try '--help'.
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
content on  main  o/  

-----------------------------------------------------------------------
fsh$ yarn content --help

yarn run v1.22.22
$ yarn -s info:rari && env-cmd --silent cross-env CONTENT_ROOT=files rari-tool --help

🐥 This command is now using rari: https://github.com/mdn/rari
🐞 Please report any issues here:  https://github.com/mdn/rari/issues/new?template=bug.yml

Subcommands for altering content programmatically

Usage: rari content [OPTIONS] <COMMAND>

Commands:
  move                     Moves content pages from one slug to another
  delete                   Deletes content pages
  add-redirect             Adds a redirect from->to pair to the redirect map
  sync-translated-content  Syncs translated content for all or a list of locales
  fmt-sidebars             Formats all sidebars
  sync-sidebars            Sync sidebars with redirects
  fix-redirects            Fixes redirects across all locales
  validate-redirects       Validate redirects
  inventory                Create content inventory as JSON
  help                     Print this message or the help of the given subcommand(s)

Options:
  -v, --verbose...  Increase logging verbosity
  -q, --quiet...    Decrease logging verbosity
  -h, --help        Print help
  -V, --version     Print version
Done in 0.22s.

@OnkarRuikar OnkarRuikar requested a review from a team as a code owner February 26, 2025 04:08
@github-actions github-actions bot added system [PR only] Infrastructure and configuration for the project size/xs [PR only] 0-5 LoC changed labels Feb 26, 2025
@Josh-Cena
Copy link
Member

@mdn/core-dev Is there interest in implementing this soon?

@OnkarRuikar OnkarRuikar requested a review from caugner March 7, 2025 01:18
Copy link
Member

@bsmth bsmth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we're good to go given #38332 (comment), so leaving a +1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/xs [PR only] 0-5 LoC changed system [PR only] Infrastructure and configuration for the project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants