Skip to content

Conversation

@fabianfett
Copy link
Member

@fabianfett fabianfett commented Jun 16, 2023

Motivation

We want to use the swift-server CI going forward. For this we need to add a bunch of docker magic to work.

Changes

  • Add docker-compose files to run tests in swift-server ci

Open ends

Some of the CI jobs will fail. We should enable it and one by one make it green. First one is already posted: #52.

@fabianfett fabianfett added this to the Transfer milestone Jun 16, 2023
@Joannis
Copy link
Member

Joannis commented Jun 16, 2023

@fabianfett how do builds get triggered?

@fabianfett fabianfett added the semver/none No version bump required. label Jun 16, 2023
@yim-lee
Copy link
Collaborator

yim-lee commented Jun 16, 2023

@swift-server-bot test this please

@yim-lee
Copy link
Collaborator

yim-lee commented Jun 16, 2023

@Joannis CI should be triggered automatically once I set them up, or they can be force-triggered using "test this please" comment (e.g., #54 (comment))

@fabianfett
Copy link
Member Author

CI fails as expected. Will fix with follow up PR #52.

Copy link
Collaborator

@yim-lee yim-lee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI jobs are configured. We can add soundness check job etc. later when ready. Just let me know.

Comment on lines +31 to +34
ARG swiftformat_version=0.48.8
RUN git clone --branch $swiftformat_version --depth 1 https://github.com/nicklockwood/SwiftFormat $HOME/.tools/swift-format
RUN cd $HOME/.tools/swift-format && swift build -c release
RUN ln -s $HOME/.tools/swift-format/.build/release/swiftformat $HOME/.tools/swiftformat
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can remove this block if there is no plan to run swiftformat.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I intend to enable it very soon :)

@fabianfett fabianfett merged commit a7d196e into main Jun 16, 2023
@fabianfett fabianfett deleted the ff-ci branch June 16, 2023 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver/none No version bump required.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants