swagger-api / swagger-api/swagger-codegen
Review isDataTypeFile, isDataTypeBinary in all generators
Nobody has claimed this yet.
- Dominant language
- Mustache
- Stars
- 17.8k
- Forks
- 6k
- PR merge metrics
- No merged PRs in 30d
Description
Description
Similar to https://github.com/swagger-api/swagger-codegen/issues/4764, we'll need to review other generators to make sure isDataTypeFile and isDataTypeBinary are properly overridden.
Swagger-codegen version
Latest master
Suggest a Fix
The fix should be similar to https://github.com/swagger-api/swagger-codegen/pull/5065
If anyone wants to help on this, please reply to let us know.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the generator implementations of isDataTypeFile and isDataTypeBinary, then compare the expected approach with issue 4764 and pull request 5065. Done means every affected generator has been reviewed and the necessary overrides are consistent with that reference.
Written by the indexing model from the issue text.
Assessment
- Domain
- devtools
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100