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

feat: Add push and pull options for the android-app image #3

Merged
merged 1 commit into from
Nov 25, 2022

Conversation

angonz
Copy link
Contributor

@angonz angonz commented Jun 29, 2022

Add tutor images push and tutor images push options for the android-app image.

tutorandroid/plugin.py Outdated Show resolved Hide resolved
tutorandroid/plugin.py Outdated Show resolved Hide resolved
tutorandroid/plugin.py Outdated Show resolved Hide resolved
Copy link
Contributor

@regisb regisb left a comment

Choose a reason for hiding this comment

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

I detected relatively obvious errors which I had not seen earlier... Please take the time to thoroughly and patiently test this change :)

tutorandroid/plugin.py Outdated Show resolved Hide resolved
tutorandroid/plugin.py Outdated Show resolved Hide resolved
@regisb
Copy link
Contributor

regisb commented Sep 8, 2022

@angonz did you see my comments above?

@angonz
Copy link
Contributor Author

angonz commented Sep 8, 2022

Hi! Yes, I'm sorry I didn't have time to make the changes yet.

@regisb
Copy link
Contributor

regisb commented Nov 15, 2022

@angonz will you have time to make the changes in time for Olive?

@angonz
Copy link
Contributor Author

angonz commented Nov 24, 2022

Hi @regisb, sorry for the delay. I have made the suggested changes and tested in my k8s site. Images build and push correctly, and the app.apk can be downloaded (although I have not tested it)

Copy link
Contributor

@regisb regisb left a comment

Choose a reason for hiding this comment

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

Thanks Andrés! Can you please squash your commits and not bump the plugin version?

tutorandroid/__about__.py Outdated Show resolved Hide resolved
@angonz angonz force-pushed the angonz/push-app-images branch 4 times, most recently from 3cbe923 to 443ac70 Compare November 25, 2022 12:33
@regisb regisb merged commit 5fceb02 into overhangio:master Nov 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

2 participants