VishnuSanal / Quotes

Quotes Status Creator lets you share quotations as status images on social media
https://play.google.com/store/apps/details?id=phone.vishnu.quotes
GNU General Public License v3.0
50 stars 20 forks source link

Fastlane structures implemented #68

Closed VishnuSanal closed 2 years ago

VishnuSanal commented 2 years ago

Ping @IzzySoft

Also, the screenshots are not added right now. I have nothing to change, still should those be added? Will this break the current screenshots that's live right now?

I tried adding the same ones I use for Play Store, but the tests fail & shows issues with aspect ratio. So, is it okay to have fastlane structures without screenshots & feature graphic?

Or should I download the current assets that are live here and add it to the repo?

Waiting for your confirmation to create the tag. Thanks! :)

IzzySoft commented 2 years ago

Will this break the current screenshots that's live right now?

There won't be any screenshots at F-Droid as there are none in your Fastlane structure. I don't know nothing about what your tests do (I'm not even an Android dev), so I cannot tell.

The minimum Fastlane is set up and acceptable by F-Droid, you even built upon it and added some more – so for your RFP it should be sufficient. But without screenshots in Fastlane there won't be screenshots in F-Droid.

Looking at the workflow errors: Did you really try to add thumbnails instead of screenshots? 189px is indeed quite small. No idea with the featureGraphic, though.

VishnuSanal commented 2 years ago

I don't know nothing about what your tests do (I'm not even an Android dev), so I cannot tell.

I meant this GitHub action to validate fastlane fails (that I saw you mention here) - sorry for not clarifying.

The minimum Fastlane is set up and acceptable by F-Droid, you even built upon it and added some more – so for your RFP it should be sufficient. But without screenshots in Fastlane there won't be screenshots in F-Droid.

I was mentioning about IzzyOnDroid. This app is not yet eligible for F-Droid yet.

Looking at the workflow errors: Did you really try to add thumbnails instead of screenshots? 189px is indeed quite small. No idea with the featureGraphic, though.

Error: 'max:min' edge radio should be at most 2.0: got=2.14

I got this error at first, so I was trying to fix it ;)

Anyways, I added screenshots & feature graphic back, the workflow doesn't seem to accept those now too (gives the same error). Will this be a problem for your (IzzyOnDroid's) indexer to pull from my repo?

Edit: Or can I just ignore the workflow's result?

Thanks! - Vishnu

IzzySoft commented 2 years ago

Ah, now. OK, consider those Fastlane checkers you use with the Github actions as running in "strict mode", while we are happy with "compatibility mode". My repo has an "upper cap", so the process pulling files from Fastlane takes care to downsize if larger (smallest side not wider than 350px, compress, if needed convert to JPG). So no worries. Not sure what F-Droid itself checks, but there it's rather my downsizing being skipped than more checks being added.

I was mentioning about IzzyOnDroid. This app is not yet eligible for F-Droid yet.

Adding it right now. Some notes while checking:

Or can I just ignore the workflow's result?

You probably can, yes, and most people won't notice. They seem to be good guidance, though: An aspect ratio higher than 1:2 is hard to look at (looks unnatural).

shuvashish76 commented 2 years ago

izzyrepo missed latest v2.7.0. Some Fastlane issues @IzzySoft ?

IzzySoft commented 2 years ago

No. Some naming issue, check your tag (hint: CaseSensitivity).

VishnuSanal commented 2 years ago

No. Some naming issue, check your tag (hint: CaseSensitivity).

Oh, yes. Updated now.

izzyrepo missed latest v2.7.0. Some Fastlane issues @IzzySoft ?

Thanks for noticing that & reporting it :)

IzzySoft commented 2 years ago

Then v2.7.0 should show up in about an hour :smiley:

IzzySoft commented 2 years ago

And there it is :smile: