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 unknown orientation variant #411

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

GeorgeElsham
Copy link

Fixes #410

@GeorgeElsham
Copy link
Author

@Zandor300 I don't believe any of the CI should fail, and it appeared to fail when trying to download Ruby. Is it worth you re-running the CI?

@Zandor300
Copy link
Member

@GeorgeElsham Is it possible for you to update your PR from master? I've fixed the CI test jobs. Someone removed the ruby version we were using and that is fixed on master.

Danger job is broken when running on a PR that merges from a fork so don't mind that.

@GeorgeElsham
Copy link
Author

I've not really dealt with using forks and pull requests on GitHub. I used GitHub's "Sync fork" button but it seems to have merged your master into my PR branch. If this is okay, we can continue. Otherwise, let me know if I should rebase instead.

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.

Orientation incorrectly reported as portrait
2 participants