Skip to content

🔥 Cleanup: Remove unused templates and simplify signatures#178

Open
MadsBogeskov wants to merge 1 commit intoios-434-convert-apirequest-generator-to-stencil-templatefrom
ios-435-cleanup-remove-old-toswift-methods-static-files-and-inline
Open

🔥 Cleanup: Remove unused templates and simplify signatures#178
MadsBogeskov wants to merge 1 commit intoios-434-convert-apirequest-generator-to-stencil-templatefrom
ios-435-cleanup-remove-old-toswift-methods-static-files-and-inline

Conversation

@MadsBogeskov
Copy link
Contributor

@MadsBogeskov MadsBogeskov commented Mar 13, 2026

Problem

After completing the Stencil template migration (IOS-427 through IOS-434), some unused artifacts and dead code remain in the codebase.

Solution

  • Delete unused DummyTest.stencil template (not referenced anywhere)
  • Remove unused swaggerFile and accessControl parameters from GlobalHeadersModel.toSwift() — these were leftover from before the Stencil migration and are not used in the method body
  • Remove unused import SwaggerSwiftML from GlobalHeadersModel.swift

Note: The Static Files/ directory was already removed in PR #170. Helpers like indentLines(), addNewlinesIfNonEmpty(), and documentationFormat() are still actively used for building pre-rendered body strings passed to Stencil templates, so they are intentionally kept.

Testing

All 70 tests pass, including golden file tests (byte-for-byte identical output).

Stack

#169#170#171#172#173#174#175#176#177 → this PR

IOS-435


Note

Low Risk
Low risk cleanup that only removes unused template code and drops unused parameters/imports in GlobalHeadersModel.toSwift(), with minimal behavioral impact aside from a signature change.

Overview
Removes dead Stencil scaffolding by deleting the unused DummyTest.stencil template.

Simplifies GlobalHeadersModel.toSwift() by dropping unused swaggerFile/accessControl parameters and removing an unused SwaggerSwiftML import, updating Generator to call the new signature when emitting GlobalHeaders.swift.

Written by Cursor Bugbot for commit a2d0ade. This will update automatically on new commits. Configure here.

@MadsBogeskov MadsBogeskov marked this pull request as ready for review March 16, 2026 12:37
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.

1 participant