diff options
| author | Xe Iaso <me@xeiaso.net> | 2025-03-28 14:55:25 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-03-28 14:55:25 -0400 |
| commit | 937f1dd330294801411b78ab497073f36ebedd47 (patch) | |
| tree | 7ff7052a1d3721970abb655dcec191a510b28b5b /PULL_REQUEST_TEMPLATE.md | |
| parent | bb4f49cfd94783111e2fbed99b4ea7a2077fa0bf (diff) | |
| download | anubis-937f1dd330294801411b78ab497073f36ebedd47.tar.xz anubis-937f1dd330294801411b78ab497073f36ebedd47.zip | |
all: do not commit generated JS/CSS to source control (#148)
Closes #125
Closes #40
Among other things, this moves all of the asset generation to run within
the context of an npm script. Developer documentation stubs have been
added so that people can get started more easily.
The top-level Dockerfile (which is no longer used in production) has
been removed as its presence has been causing confusion. This changeset
will break it anyways.
These changes will make for less "repo churn" as the static assets are
built and rebuilt, at the cost of making the build step more complicated
for downstream packagers. If this becomes a burden, we can explore
making a "release tarball" that contains pre-massaged outputs.
Diffstat (limited to 'PULL_REQUEST_TEMPLATE.md')
| -rw-r--r-- | PULL_REQUEST_TEMPLATE.md | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/PULL_REQUEST_TEMPLATE.md b/PULL_REQUEST_TEMPLATE.md index 78992fa..e0ebdcb 100644 --- a/PULL_REQUEST_TEMPLATE.md +++ b/PULL_REQUEST_TEMPLATE.md @@ -1,6 +1,11 @@ -<!-- delete me and describe your change here --> +<!-- +delete me and describe your change here, give enough context for a maintainer to understand what and why + +See https://anubis.techaro.lol/docs/developer/code-quality for more information +--> Checklist: - [ ] Added a description of the changes to the `[Unreleased]` section of docs/docs/CHANGELOG.md -- [ ] Tested this at least manually +- [ ] Added test cases to [the relevant parts of the codebase](https://anubis.techaro.lol/docs/developer/code-quality) +- [ ] Ran integration tests `npm run test:integration` |