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

ci: Add an action to upload APKs to Waldo #4272

Merged
merged 2 commits into from
Jul 15, 2023
Merged

Conversation

g123k
Copy link
Collaborator

@g123k g123k commented Jul 2, 2023

The code is now really to upload APKs (IPA will come once we are confident this one works well).

It will only be with PRs in a first time, and also we can't have a direct link to the website to test, as the Waldo session doesn't provide what's necessary yet (a bug is opened on their side)

@github-actions github-actions bot added the GitHub label Jul 2, 2023
@g123k g123k linked an issue Jul 2, 2023 that may be closed by this pull request
@codecov-commenter
Copy link

codecov-commenter commented Jul 2, 2023

Codecov Report

Merging #4272 (1cfc41f) into develop (bb74c5a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #4272   +/-   ##
========================================
  Coverage    10.87%   10.87%           
========================================
  Files          285      285           
  Lines        14170    14170           
========================================
  Hits          1541     1541           
  Misses       12629    12629           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@g123k g123k changed the title [WIP] Add an action to upload APKs to Waldo feat: Add an action to upload APKs to Waldo Jul 2, 2023
@g123k g123k marked this pull request as ready for review July 2, 2023 22:30
@g123k g123k requested a review from a team as a code owner July 2, 2023 22:30
with:
#channel: stable
cache: true
flutter-version: ${{ steps.flutter-version.outputs.FLUTTER_VERSION }}
Copy link
Member

Choose a reason for hiding this comment

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

You forgot to include the step which outputs the flutter version we want

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ooops you're right, that's a bad copy/paste.
That should now be OK (line 28)

Copy link
Member

@M123-dev M123-dev left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks

Looking foreward to testing this out

@g123k
Copy link
Collaborator Author

g123k commented Jul 15, 2023

Thank you @M123-dev, let's try to merge it

@g123k g123k merged commit 66ba6cd into openfoodfacts:develop Jul 15, 2023
@g123k g123k deleted the waldo branch July 15, 2023 05:59
@g123k g123k changed the title feat: Add an action to upload APKs to Waldo ci: Add an action to upload APKs to Waldo Aug 5, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Aug 6, 2023

You can test this PR on: Android

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Announcement] We will test Waldo Sessions on PRs
3 participants