Skip to content

Add .toml and .cff rules to .editorconfig, and clean up file comments#7994

Merged
mhucka merged 6 commits intoquantumlib:mainfrom
mhucka:add-editorconfig-toml
Apr 4, 2026
Merged

Add .toml and .cff rules to .editorconfig, and clean up file comments#7994
mhucka merged 6 commits intoquantumlib:mainfrom
mhucka:add-editorconfig-toml

Conversation

@mhucka
Copy link
Copy Markdown
Contributor

@mhucka mhucka commented Mar 31, 2026

This adds a missing indentation rule for .toml files, and adds .cff to the YAML section.

Also, this PR removes needless comments and comment verbosity.

This adds a missing indentation rule for `.toml` files.

Also, this removes needless comments.
@github-actions github-actions bot added the size: S 10< lines changed <50 label Mar 31, 2026
@mhucka mhucka marked this pull request as ready for review March 31, 2026 00:05
@mhucka mhucka requested review from a team and vtomole as code owners March 31, 2026 00:05
@mhucka mhucka requested a review from dabacon March 31, 2026 00:05
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 31, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.63%. Comparing base (aa9a4da) to head (cf6b071).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7994      +/-   ##
==========================================
- Coverage   99.63%   99.63%   -0.01%     
==========================================
  Files        1110     1110              
  Lines       99749    99749              
==========================================
- Hits        99388    99387       -1     
- Misses        361      362       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@pavoljuhas pavoljuhas added the ci/no-release Use this label for pull request that should not have Cirq pre-release on PyPI. label Apr 3, 2026
@mhucka mhucka added this pull request to the merge queue Apr 4, 2026
Merged via the queue into quantumlib:main with commit c7c054e Apr 4, 2026
72 of 75 checks passed
@mhucka mhucka deleted the add-editorconfig-toml branch April 4, 2026 06:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/no-release Use this label for pull request that should not have Cirq pre-release on PyPI. size: S 10< lines changed <50

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants