| Commit message (Collapse) | Author | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Poetry 1.2 introduced a regression which broke pip `--user` installs.
These types of install where the main way we did installations in docker
and CI, which made it much more convenient to control their location,
availability, and caching.
Poetry's team does not recognize this as a supported use case, so major
changes were required to get everything working again. Most of the
changes were consolidated into chrislovering/python-poetry-base for
docker, and hassanabouelela/setup-python for CI.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
Whitenoise raises a warning when the static content folder does not
exist, which is the case during tests in CI. This is not an issue
though, and static content does get used properly in tests. Thus, the
warning is silenced.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
There are currently no warnings emitted during our tests, so this is a
low-cost method of ensuring we can better notice problems such as
deprecation warnings earlier.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
Removes a few features which were deprecated in django 4.0. Running with
warnings enabled shows no other errors.
USE_L10N was deprecated, and is now enabled by default. In future
versions of django, it'll be impossible to turn localization off.
Explicitly defining the custom_app_config for the API app is no longer
necessary as django can pick it up on its own.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
|
|
In reference to issue #2108, this commit aims to add an initial set of keywords per rule.
These keywords will be later in the "rule" bot command in order to make rule identification easier
|
|
|
|
The infraction tests checked that the route returned infractions in the
correct order, which is based on insertion time. This can be fragile
however, since the insertion time can be very close (or identical)
during the tests. That became especially more likely with PR #741
(commit 149e67b4) which improved database access speed.
This is fixed by explicitly specifying the insertion time, and spacing
them out properly.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
Sets the timeout of downloads to 3 minutes (from a default of 5 seconds)
since some artifacts can be quite large.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
Adds a missing raise in the static build for situations where everything
was successful, but the build time passed the deadline specified by the
site API.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
|
|
|
|
A default is set for backwards compatibility with bot version that don't explicitly give a value.
|
|
When reading the created_at time from the GitHub API, it'll be a naive
date string with UTC time, so we use that instead of the system's time.
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
|
|
Co-authored-by: wookie184 <[email protected]>
|
|
Co-authored-by: wookie184 <[email protected]>
|
|
|
|
|
|
|
|
|
|
Before this change, the rule technically disallows implementing JavaScript clients because webbrowsers use HTTP to fetch the static files.
This commit also fixes a mistake in a previous commit where Markdown-style characters were used instead of code-tags.
|
|
Technically it's an ongoing jam, not a previous jam. But I think adding this link will help with discoverability if you're already on the CJ Info page. Can be updated once a theme is announced
|
|
This was the easiest way of navigating to the CJ9 page previously other than the home page. Let's keep it for the duration of the jam
|
|
|
|
|
|
|
|
Co-authored-by: wookie184 <[email protected]>
|
|
Co-authored-by: Kieran Siek <[email protected]>
|
|
|
|
|
|
|
|
Signed-off-by: Hassan Abouelela <[email protected]>
|
|
Co-authored-by: Numerlor <[email protected]>
|
|
|
|
|
|
|
|
|
|
Moves the responsibility of re-requesting a workflow run from the API to
the user. This makes the requests much shorter-lived, and allows the
client to control how they want to handle sleeping and retrying.
This also has the benefit of removing the only real piece of async code,
so now the view is completely sync once again.
Signed-off-by: Hassan Abouelela <[email protected]>
|