Skip to content

Enable beman-tidy default mode#55

Closed
mariobullet wants to merge 1 commit intobemanproject:mainfrom
mariobullet:issue/259-default-mode
Closed

Enable beman-tidy default mode#55
mariobullet wants to merge 1 commit intobemanproject:mainfrom
mariobullet:issue/259-default-mode

Conversation

@mariobullet
Copy link
Copy Markdown

@mariobullet mariobullet commented Apr 25, 2026

issue: bemanproject/beman-tidy#259
Summary

  • enable beman-tidy in default mode via pre-commit
  • add the project LICENSE file and README license note
  • rename sorted_list.pass.cpp to sorted_list.test.cpp and update the test target

Comment thread LICENSE
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This file should be discarded.

This repo already has a license file, but with WRONG naming - LICENSE.txt should be renamed to LICENSE (https://github.com/bemanproject/beman/blob/main/docs/beman_standard.md#toplevellicense).

More over, since we actually use APACHE license, please copy exact file from https://github.com/bemanproject/beman/blob/main/docs/beman_standard.md#licenseapache_llvm (a.k.a. https://github.com/bemanproject/beman/blob/main/LICENSE).

Comment thread README.md

## License

Source is licensed with the Apache License v2.0 with LLVM Exceptions.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please reimplement https://github.com/bemanproject/beman/blob/main/docs/beman_standard.md#readmelicense

Use exactly the following format:

## License

beman.exemplar is licensed under the Apache License v2.0 with LLVM Exceptions.

<!-- Other optional mentions go on this line ... >

TLDR: you should have sa single line beman.net is licensed under the Apache License v2.0 with LLVM Exceptions.

@neatudarius
Copy link
Copy Markdown
Member

@mariobullet , discarded this PR as it seems more work to be done then starting from scratch - I will help you as indicated in bemanproject/beman-tidy#259 (comment).

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.

2 participants