Skip to content

ci: enable beman-tidy in default mode#412

Closed
DaviP-Conceicao wants to merge 2 commits into
bemanproject:mainfrom
DaviP-Conceicao:add-beman-tidy-default
Closed

ci: enable beman-tidy in default mode#412
DaviP-Conceicao wants to merge 2 commits into
bemanproject:mainfrom
DaviP-Conceicao:add-beman-tidy-default

Conversation

@DaviP-Conceicao

Copy link
Copy Markdown

Summary

Enables beman-tidy in default mode as requested in bemanproject/beman-tidy#330.

Findings

Ran beman-tidy in default mode across the target library (exemplar) using pre-commit. All checks completed successfully with zero errors (Passed), so no preliminary bug-fix PRs were necessary (skipping Step 3 of the integration guide).

Comment thread .pre-commit-config.yaml Outdated
@coveralls

coveralls commented Jul 9, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 100.0%. remained the same — DaviP-Conceicao:add-beman-tidy-default into bemanproject:main

@DaviP-Conceicao

Copy link
Copy Markdown
Author

I have updated the branch to fix the Cookiecutter discrepancy and added the missing EOF newlines. All pre-commit, linting, and formatting checks are now passing! 🎉

Note to maintainers: One job in the build matrix (clang 22 c++26 libstdc++ Rele
Captura de tela_2026-07-09_19-25-53
Captura de tela_2026-07-09_19-26-58
ase.MaxSan) failed during the container initialization step with:
Error: Docker login for 'ghcr.io' failed with exit code 1

Since this appears to be an automated GITHUB_TOKEN / registry permission restriction for pull requests originating from external forks, could a maintainer please re-run that specific job or review the PR? Thank you!

@neatudarius

Copy link
Copy Markdown
Member

We previously decided to not integrate beman-tidy inside exemplar, but run an exemplar-based test inside beman-tidy repo: https://github.com/bemanproject/beman-tidy/blob/main/.github/workflows/run-on-exemplar.yml. I will discard this PR

CC: @ednolan

@neatudarius neatudarius closed this Jul 9, 2026
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.

3 participants