glideapps/quicktype

Swift 5.9 warning about Hashable being deprecated

Open

#2 423 ouverte le 11 oct. 2023

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)TypeScript (10 867 stars) (968 forks)batch import
Swiftgood first issue

Description

Generate swift code has compilation warning about deprecated protocol.

SamSchema.swift:5645:16: warning: 'Hashable.hashValue' is deprecated as a protocol requirement; conform type 'JSONNull' to 'Hashable' by implementing 'hash(into:)' instead
    public var hashValue: Int {

Using :

swift --version 
swift-driver version: 1.87.1 Apple Swift version 5.9 (swiftlang-5.9.0.128.108 clang-1500.0.40.1)
Target: arm64-apple-macosx14.0

Steps to reproduce :

curl -o sam.schema.json https://raw.githubusercontent.com/aws/serverless-application-model/develop/schema_source/sam.schema.json

quicktype -s schema sam.schema.json -o SamSchema.swift

swift SamSchema.swift 
SamSchema.swift:5645:16: warning: 'Hashable.hashValue' is deprecated as a protocol requirement; conform type 'JSONNull' to 'Hashable' by implementing 'hash(into:)' instead
    public var hashValue: Int {

Expected behaviour : generated code is updated to not include deprecated features.

Thanks !

Guide contributeur

Stack technique
swifttypescript
Domaine
tooling
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Basic Swift knowledgeFamiliarity with quicktype's codebase
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80
Direction de recherche
The issue is about updating Swift code generation to replace deprecated 'Hashable.hashValue' with 'hash(into:)'. Look at quicktype's Swift code generation templates, likely in src/language/Swift. Find where JSONNull conforms to Hashable and implement the hash(into:) method. Test by generating Swift code from the SAM schema and verifying no warnings.