Skip to content

Add support for boolean value for color option#244

Merged
sindresorhus merged 4 commits intosindresorhus:mainfrom
Mallikarjun-0:Minor-change-in-index.d.ts
Feb 2, 2025
Merged

Add support for boolean value for color option#244
sindresorhus merged 4 commits intosindresorhus:mainfrom
Mallikarjun-0:Minor-change-in-index.d.ts

Conversation

@Mallikarjun-0
Copy link
Copy Markdown
Contributor

@Mallikarjun-0 Mallikarjun-0 commented Jan 28, 2025

color property of class Ora can be a boolean false for the default color of spinner, but the type hint only mentions Color whereas it should be Color | boolean.

@sindresorhus
Copy link
Copy Markdown
Owner

You have not updated all the places that needs updating.

@Mallikarjun-0
Copy link
Copy Markdown
Contributor Author

@sindresorhus I changed the type in the options as well, is there any other place where it needs to be changed?

@sindresorhus
Copy link
Copy Markdown
Owner

I changed the type in the options as well, is there any other place where it needs to be changed?

I think if you just checked instead of asking you would quickly see that you also need to update the readme.

@Mallikarjun-0
Copy link
Copy Markdown
Contributor Author

@sindresorhus updated the readme with the required type changes.

@sindresorhus sindresorhus changed the title Ora.color can have a boolean value false for default spinner color. Add support for boolean value for color option Feb 2, 2025
@sindresorhus sindresorhus merged commit fe951e5 into sindresorhus:main Feb 2, 2025
3 checks passed
@Mallikarjun-0 Mallikarjun-0 deleted the Minor-change-in-index.d.ts branch April 18, 2025 10:32
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.

2 participants