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

Use download latest script in CI #216

Merged
merged 2 commits into from
Oct 18, 2021
Merged

Use download latest script in CI #216

merged 2 commits into from
Oct 18, 2021

Conversation

curquiza
Copy link
Member

@curquiza curquiza commented Oct 16, 2021

Since meilisearch/meilisearch#1655 has been fixed into the download-latest.sh script, we can use it in the CI

@curquiza
Copy link
Member Author

bors try

bors bot added a commit that referenced this pull request Oct 16, 2021
@bors
Copy link
Contributor

bors bot commented Oct 16, 2021

try

Build failed:

@curquiza
Copy link
Member Author

bors try

bors bot added a commit that referenced this pull request Oct 16, 2021
@bors
Copy link
Contributor

bors bot commented Oct 16, 2021

try

Build failed:

@curquiza
Copy link
Member Author

Incredible! It has worked the first time:
Capture d’écran 2021-10-16 à 17 39 36

(I re-run the workflow)

But now, it does not!

@curquiza
Copy link
Member Author

bors try

bors bot added a commit that referenced this pull request Oct 16, 2021
@bors
Copy link
Contributor

bors bot commented Oct 16, 2021

try

Build succeeded:

@curquiza
Copy link
Member Author

bors try

bors bot added a commit that referenced this pull request Oct 16, 2021
@bors
Copy link
Contributor

bors bot commented Oct 16, 2021

try

Build succeeded:

@curquiza
Copy link
Member Author

bors try

bors bot added a commit that referenced this pull request Oct 16, 2021
@bors
Copy link
Contributor

bors bot commented Oct 16, 2021

try

Build succeeded:

@curquiza
Copy link
Member Author

@bidoubiwa looks like the CI works now 🎉

@curquiza curquiza marked this pull request as ready for review October 16, 2021 16:12
@curquiza
Copy link
Member Author

bors try

bors bot added a commit that referenced this pull request Oct 18, 2021
@bors
Copy link
Contributor

bors bot commented Oct 18, 2021

try

Build succeeded:

Copy link
Contributor

@bidoubiwa bidoubiwa left a comment

Choose a reason for hiding this comment

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

LGTM 🔥

@bidoubiwa
Copy link
Contributor

bors merge

@bors
Copy link
Contributor

bors bot commented Oct 18, 2021

Build succeeded:

@bors bors bot merged commit bf2aef5 into main Oct 18, 2021
@bors bors bot deleted the use-download-latest-in-CI branch October 18, 2021 10:05
bors bot added a commit to meilisearch/meilisearch that referenced this pull request Oct 19, 2021
1822: Tiny improvements in download-latest.sh r=irevoire a=curquiza

- Add check on `$latest` to check if it's empty. We have some issue on the swift SDK currently where the version number seems not to be retrieved, but we don't why meilisearch/meilisearch-swift#216
- Replace some `"` by `'`
- Rename `$BINARY_NAME` by `$binary_name` to make them consistent with the other variables that are filled all along the script

Co-authored-by: Clémentine Urquizar <clementine@meilisearch.com>
@bidoubiwa bidoubiwa added the skip-changelog The PR will not appear in the release changelogs label Nov 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog The PR will not appear in the release changelogs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants