Closed ghost closed 6 months ago
Also ideally the PR title should be using the conventional commits style.
@uncenter @isabelroses i added the code that you guys asked for
Also ideally the PR title should be using the conventional commits style.
maybe
Also ideally the PR title should be using the conventional commits style.
We can fix this at merge time, no big deal. This is going to get squashed anyways.
@nullishamy @uncenter @isabelroses i fixed the issues
@nullishamy @uncenter @isabelroses i fixed the issues
Instead of re-pinging us there is a button under the "Reviewers" tab on the right panel that lets you re-request a review.
@nullishamy @uncenter @isabelroses i fixed the issues
Instead of re-pinging us there is a button under the "Reviewers" tab on the right panel that lets you re-request a review.
oh yeah i forgot about that
This needs merged after #178, so lets wait for that. Otherwise LGTM.
alright
Freezing this PR, no more changes are to be made. We will rework the README further once we release the new major. Subsequent change requests will be ignored.
Freezing this PR, no more changes are to be made. We will rework the README further once we release the new major. Subsequent change requests will be ignored.
alr
this PR improves and fixes the readme for the new build system