Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: enable Swift CI tests #2838

Merged
merged 2 commits into from
Jul 7, 2020
Merged

Conversation

FUSAKLA
Copy link
Member

@FUSAKLA FUSAKLA commented Jul 2, 2020

Signed-off-by: Martin Chodur [email protected]

Added Swift CI tests as suggested in #2829

I'm not sure if it should run for every test? I'm not sure if we are somehow limited in the CircleCI usage.
From time perspective it does not significantly prolong the test stage in CI.

@FUSAKLA FUSAKLA force-pushed the fus-swift-ci-tests branch 6 times, most recently from 1e20edd to b97fca7 Compare July 2, 2020 23:07
Signed-off-by: Martin Chodur <[email protected]>
@FUSAKLA FUSAKLA force-pushed the fus-swift-ci-tests branch from b97fca7 to 158b13d Compare July 2, 2020 23:25
@FUSAKLA FUSAKLA marked this pull request as ready for review July 2, 2020 23:25
@FUSAKLA
Copy link
Member Author

FUSAKLA commented Jul 2, 2020

@GiedriusS PTAL

Copy link
Member

@bwplotka bwplotka left a comment

Choose a reason for hiding this comment

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

It looks good. Yes, every commit is the way we do it right now! (:

.circleci/config.yml Outdated Show resolved Hide resolved
Co-authored-by: Bartlomiej Plotka <[email protected]>
Signed-off-by: Giedrius Statkevičius <[email protected]>
@GiedriusS GiedriusS force-pushed the fus-swift-ci-tests branch from 8995cd2 to f4d32ac Compare July 7, 2020 16:48
@bwplotka
Copy link
Member

bwplotka commented Jul 7, 2020

Flaky test, rerun

Copy link
Member

@GiedriusS GiedriusS left a comment

Choose a reason for hiding this comment

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

LGTM! Added @bwplotka's suggestion to your branch and then had to force push because, again, I fell into the trap of clicking "Commit suggestion" but without adding DCO 🤦

@GiedriusS GiedriusS merged commit 7743fa0 into thanos-io:master Jul 7, 2020
GiedriusS added a commit to GiedriusS/thanos that referenced this pull request Jul 7, 2020
Added by thanos-io#2838.

Signed-off-by: Giedrius Statkevičius <[email protected]>
@FUSAKLA
Copy link
Member Author

FUSAKLA commented Jul 8, 2020

Sorry for the lag I was on vacation. Thanks for fixing the typo and merging! :)

GiedriusS added a commit that referenced this pull request Jul 8, 2020
Added by #2838.

Signed-off-by: Giedrius Statkevičius <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants