Skip to content

Rename the bin/dart_skills_lint.dart file to cli and update documenation. - #73

Merged
reidbaker merged 2 commits into
flutter:mainfrom
reidbaker:r-bin-file-name-to-cli
Apr 10, 2026
Merged

reidbaker merged 2 commits into
flutter:mainfrom
reidbaker:r-bin-file-name-to-cli

Conversation

@reidbaker

Copy link
Copy Markdown
Contributor

I thought there were too many files named dart_skils_lint.* this removes one.

Pre-launch Checklist

  • I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
  • I read the [Tree Hygiene] wiki page, which explains my responsibilities.
  • I read the [Flutter Style Guide] recently, and have followed its advice.
  • I signed the [CLA].
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is [test-exempt].
  • All existing and new tests are passing.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request renames the main CLI entry point from 'bin/dart_skills_lint.dart' to 'bin/cli.dart' across the project, including documentation, pubspec configuration, and all integration tests. I have no feedback to provide as the changes are consistent and correctly applied.

@reidbaker
reidbaker merged commit 56e9036 into flutter:main Apr 10, 2026
7 checks passed
@reidbaker
reidbaker deleted the r-bin-file-name-to-cli branch April 10, 2026 19:13
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

Successfully merging this pull request may close these issues.

1 participant