Clean up KTX/CRN support
Open
type - cleanup
- Dominant language
- JavaScript
- Stars
- 15.7k
- Forks
- 3.9k
- Avg merge
- 4d 6h
- Merged PRs (30d)
- 34
Description
There are at least 3 places where we check if a file is KTX, CRN, or a standard web image. This could probably be unified.
We are also missing tests in these places, so once we unify them we could also add a test for it. For example, ImagereyProvider.loadImage does not test KTX/CRN loading.
Contributor guide
Research direction
Start by locating the three file-type checks mentioned in the issue and read ImagereyProvider.loadImage, which is specifically noted as lacking KTX/CRN loading tests. Unify the checks and add coverage for the supported image types, including KTX and CRN, then run the affected test suite to confirm the behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- web-dev
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100