Skip to content

Fix issues in forc add/remove --help#7558

Merged
ironcev merged 2 commits intomasterfrom
ironcev/help-for-forc-add-and-remove
Feb 18, 2026
Merged

Fix issues in forc add/remove --help#7558
ironcev merged 2 commits intomasterfrom
ironcev/help-for-forc-add-and-remove

Conversation

@ironcev
Copy link
Member

@ironcev ironcev commented Feb 16, 2026

Description

Fixes #7557 by adding doc-comment to add and remove commands and removing typos and wrong comments.

Checklist

  • I have linked to any relevant issues.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have updated the documentation where relevant (API docs, the reference, and the Sway book).
  • I have added tests that prove my fix is effective or that my feature works.
  • I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
  • I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
  • I have requested a review from the relevant team or maintainers.

@ironcev ironcev self-assigned this Feb 16, 2026
@ironcev ironcev added the forc label Feb 16, 2026
@ironcev
Copy link
Member Author

ironcev commented Feb 16, 2026

👍

@ironcev ironcev requested a review from a team February 16, 2026 14:26
@ironcev ironcev marked this pull request as ready for review February 16, 2026 15:05
@ironcev ironcev requested a review from a team as a code owner February 16, 2026 15:05
@cursor
Copy link

cursor bot commented Feb 16, 2026

PR Summary

Low Risk
Help/documentation-only string and comment changes; no behavior or dependency-modification logic is touched.

Overview
Fixes forc add/forc remove CLI help text by correcting example descriptions (e.g., "Add a dependencies" -> "Add a dependency", and "Add" -> "Remove" in remove).

Replaces misleading inline comments with rustdoc doc comments for both commands so --help/generated docs describe modifying Forc.toml dependencies accurately.

Written by Cursor Bugbot for commit d55bd5b. This will update automatically on new commits. Configure here.

@ironcev ironcev enabled auto-merge (squash) February 16, 2026 15:05
@JoshuaBatty JoshuaBatty requested review from a team February 16, 2026 22:48
@ironcev ironcev merged commit f9de545 into master Feb 18, 2026
43 checks passed
@ironcev ironcev deleted the ironcev/help-for-forc-add-and-remove branch February 18, 2026 19:02
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 this pull request may close these issues.

Forc add and remove commands do not have help and have errors in examples

3 participants

Comments