swagger-api/swagger-codegen

Review isDataTypeFile, isDataTypeBinary in all generators

Open

#5,103 opened on Mar 17, 2017

View on GitHub
 (2 comments) (0 reactions) (0 assignees)HTML (12,701 stars) (5,474 forks)batch import
Enhancement: Generalhelp wanted

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

Tech stack
java
Domain
tooling
Issue type
refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
JavaFamiliarity with swagger codegenReview PR #5065
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
45
Research direction
Start by reviewing the linked PR #5065 to understand the pattern used for fixing similar issues. Then, examine each generator in the codebase (e.g., 'modules/swagger codegen/src/main/java/io/swagger/codegen/languages/') to identify where isDataTypeFile and isDataTypeBinary are defined or overridden. Check for inconsistencies and ensure all generators have proper implementations. The goal is to bring all generators in line with the fix demonstrated in PR #5065.
Review isDataTypeFile, isDataTypeBinary in all generators · swagger-api/swagger-codegen#5103 | Good First Issue