Skip to content

Conversation

@julianbenegas
Copy link

No description provided.

@julianbenegas julianbenegas changed the title add basehub support feat: add basehub support Nov 19, 2024
@matiasperz
Copy link

🔼 upvote

Copy link
Owner

@ascorbic ascorbic left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks

Copy link
Owner

@ascorbic ascorbic left a comment

Choose a reason for hiding this comment

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

Ah, can you update the tests. It looks like the expected URLs are wrong. There are instructions in the contributing guide to running them locally

@julianbenegas
Copy link
Author

Ah, can you update the tests. It looks like the expected URLs are wrong. There are instructions in the contributing guide to running them locally

Fixed! Still not 100% sure of when fit=cover should be set. I think it'll be fine for most cases though.

@julianbenegas
Copy link
Author

hey @ascorbic, i fixed the tests here eb12cd5 💪

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.

3 participants