Skip to content
This repository was archived by the owner on Apr 8, 2026. It is now read-only.

ci: add docker build and release#198

Merged
chookity-pokk merged 1 commit into
matrix-org:masterfrom
libaurea:ci/docker
Jun 4, 2025
Merged

ci: add docker build and release#198
chookity-pokk merged 1 commit into
matrix-org:masterfrom
libaurea:ci/docker

Conversation

@libaurea

@libaurea libaurea commented Jun 3, 2025

Copy link
Copy Markdown
Contributor

Regarding issue #195, it would also be possible to release the images directly on github. Unfortunately, this would require manual user intervention for the current (and potentially future) version, but it might be better than the status quo.

@chookity-pokk

Copy link
Copy Markdown
Collaborator

The tests are failing but it seems unrelated. I just got off work so give me like an hour and I can look into this. I appreciate the help here though. The beauty of open source, ladies and gentleman ❤️

@chookity-pokk

Copy link
Copy Markdown
Collaborator

I opened up this draft PR which will hopefully fix this. What's weird is the tests are working on master when it was last pushed to but not currently (when run locally). It could be that the requirements in test-requirements.txt are allowing any version and a change has been made which is causing this to now fail.

@chookity-pokk

chookity-pokk commented Jun 4, 2025

Copy link
Copy Markdown
Collaborator

@libaurea The tests are working now, since pinning the testing package versions. Please merge master into this PR and hopefully the tests will be passing.

@chookity-pokk chookity-pokk merged commit e4a14b0 into matrix-org:master Jun 4, 2025
5 checks passed
@chookity-pokk

Copy link
Copy Markdown
Collaborator

Thanks for the help here!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants