feat: add famedly control module in docker image#239
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #239 +/- ##
==========================================
- Coverage 80.14% 80.14% -0.01%
==========================================
Files 501 501
Lines 71375 71375
Branches 10732 10732
==========================================
- Hits 57202 57201 -1
- Misses 10919 10920 +1
Partials 3254 3254 see 1 file with indirect coverage changes Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
Adds the famedly-control-synapse module to the Famedly Synapse Docker image build by introducing a new versioned build argument and extending the mod-pack build matrix used by CI.
Changes:
- Add
FCS_VERSIONbuild arg and installfamedly-control-synapseindocker/Dockerfile-famedly. - Introduce
mod020includingfcs-versionand switch the currently-built mod pack list tomod017+mod020.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
docker/Dockerfile-famedly |
Adds a new build arg and pip-install of famedly-control-synapse. |
.ci/scripts/calculate_builds.py |
Adds mod020 with fcs-version and updates which mod packs are built. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
e1df2c7 to
72d3120
Compare
d66cc78 to
d59202f
Compare
bc0c36b to
5f21efd
Compare
5f21efd to
14dbea1
Compare
14dbea1 to
e09b1a5
Compare
SYN-9
edit by jason: Added in the same testing as is done for the invite-checker and the token authenticator. Also added in building for the TIM 1.1 image to maintain consistency