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

Add "office" to buildings allowed to have a housenumber #3165

Merged
merged 1 commit into from
Aug 16, 2021
Merged

Add "office" to buildings allowed to have a housenumber #3165

merged 1 commit into from
Aug 16, 2021

Conversation

yrtimiD
Copy link
Contributor

@yrtimiD yrtimiD commented Aug 15, 2021

Minor change. From my experience office buildings always have an address.

@matkoniecz
Copy link
Member

Good idea! Have you tested this change? It is trivial, it looks right but...

Just to check: https://github.com/streetcomplete/StreetComplete/pulls?q=office+address - seems to not be rejected earlier

@yrtimiD
Copy link
Contributor Author

yrtimiD commented Aug 15, 2021

Have you tested this change? It is trivial, it looks right but...
Sincerely saying - no. Overhead of installing android studio for such a minor change is pretty big, and I'll be very surprised if something will fail in this case :) But, of course, I can test it, if you insist.
Any chance you have any external builder which can produce an apk from specific branch?

@matkoniecz
Copy link
Member

No, in such case I will simply test it. Sadly setting up Android Studio is quite obnoxious and I am not aware about some easy to use builder

@smichel17
Copy link
Member

smichel17 commented Aug 15, 2021

There is an easy-to-use external builder called @smichel17 :)

https://smichel.me/files/streetcomplete/streetcomplete-debug-daac414.apk

Copy link
Member

@matkoniecz matkoniecz left a comment

Choose a reason for hiding this comment

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

tested (there is always tiny chance of for example invisible character lurking somewhere) - and it works nicely. Thank you for the contribution!

@smichel17
Copy link
Member

smichel17 commented Aug 15, 2021

I'm very sympathetic to the setup issue. I've been putting off https://gitlab.com/fdroid/fdroidserver/-/issues/829 for almost a year now because every month or two when I find myself with both the time and motivation to work on it, getting a good python dev environment set up saps all my motivation, and then by the next time I've forgotten what I needed to do to get there. To the point where I bought some additional ram so I can just do it in a Debian vm and install everything globally so I don't have to worry about virtualenv crap. /rant

If you want to do more SC dev, ping me and I'm happy to walk you through whatever setup is needed (imo Android Studio installation is relatively easy, but if you prefer to work in the terminal, installing the standalone Android SDK is not). One of these days, someone will get the dev tools setup experience right…

@yrtimiD
Copy link
Contributor Author

yrtimiD commented Aug 15, 2021

@smichel17 , thanks!
Meanwhile I'm trying to setup github actions for the build only (despite some failed tries I found), I believe this way is a most convenient for contributors who want to do a fast fix.

@westnordost westnordost merged commit dd23ba4 into streetcomplete:master Aug 16, 2021
@mnalis
Copy link
Member

mnalis commented Aug 16, 2021

@yrtimiD For me, I just added .github/workflows/build.yml to my fork, and then Github builds debug .apk for me (you can run it manually in the Actions github tab, or automatically on pull), which I just download and install manually to test changes.

@yrtimiD
Copy link
Contributor Author

yrtimiD commented Aug 16, 2021

@mnalis, right, eventually, this was an easies way to get debug apk, just took me some time to prepare it first time.
FYI, now actions/workflows/build-debug-apk is available on upstream too, so no need to mess with this each time.

@smichel17
Copy link
Member

I tried it, then canceled it, and it sent me an email about the cancellation. Sorry if I spammed anyone else.

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.

None yet

5 participants