Skip to content

Remove formats.sh and tidy up CONTRIBUTING.md#6353

Merged
c-bata merged 15 commits intooptuna:masterfrom
sawa3030:delete-formats-shell-script-1
Nov 21, 2025
Merged

Remove formats.sh and tidy up CONTRIBUTING.md#6353
c-bata merged 15 commits intooptuna:masterfrom
sawa3030:delete-formats-shell-script-1

Conversation

@sawa3030
Copy link
Copy Markdown
Collaborator

Motivation

  • This PR is a follow-up to Migrate to ruff from black/blackdoc/isort/flake8 #6341
  • Remove formats.sh now that we’ve migrated from black/blackdoc/isort/flake8 to Ruff and no longer need a complex shell script for formatting.
  • I noticed that some command examples in CONTRIBUTING.md start with $ while others do not, so I made a small update to keep the style consistent.

Description of the changes

  • Delete formats.sh and update CONTRIBUTING.md accordingly.
  • Remove the leading $ from command examples in CONTRIBUTING.md to keep the formatting consistent across the document.

@sawa3030
Copy link
Copy Markdown
Collaborator Author

I will mark this PR as ready for review once PR #6341 has been merged.

@sawa3030 sawa3030 marked this pull request as ready for review November 21, 2025 08:20
Copy link
Copy Markdown
Member

@c-bata c-bata left a comment

Choose a reason for hiding this comment

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

LGTM.

@c-bata c-bata added the other Issue/PR that does not fit into any other label. label Nov 21, 2025
@c-bata c-bata added this to the v4.7.0 milestone Nov 21, 2025
@c-bata c-bata merged commit b7d63ad into optuna:master Nov 21, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

other Issue/PR that does not fit into any other label.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants