beberlei / assert

Thin assertion library for use in libraries and business-model
Other
2.41k stars 186 forks source link

assert/CONTRIBUTING.md on `master` #321

Open ktomk opened 2 years ago

ktomk commented 2 years ago

Wanted to file a PR for reporting an issue and followed How to contribute.

Thanks for the nice introduction in it and also for not being extensive.

But already with the second bullet-point:

  • Coding standard compliance must be ensured before committing or opening pull requests by running composer assert:cs-fix or composer assert:cs-lint in the root directory of this repository.

I run into an issue as when I execute it on a clean checkout of master, it already modifies files. And some of the fixes even don't look good to me. So just wondering if they (the composer script commands) currently apply and how to proceed. Unit tests run so I hope this ain't a blocker. Maybe it's just me or the tooling.

If interested, I've put diffs and docs into a fake PR on my fork, as both the diffs and the typescript is verbose: https://github.com/ktomk/assert/pull/1