nerzh / swift-telegram-sdk

🤖 The wrapper for the Telegram Bot API written in Swift. It's not a framework. There is no special syntax here. This is a library that implements all Telegram Bot API methods, which is available to you to work with Vapor, Smoke, Hummingbird, FlyingFox.
https://core.telegram.org/bots/api#available-methods
MIT License
203 stars 32 forks source link

Consider Declarative Syntax #16

Closed VladimirCreator closed 10 months ago

VladimirCreator commented 11 months ago

В конце файла Readme.md можно увидеть следующий отрывок кода:

// swift-tools-version:5.8

import PackageDescription

var packageDependencies: [Package.Dependency] = [
    .package(url: "https://github.com/vapor/vapor.git", .upToNextMajor(from: "4.57.0")),
]

packageDependencies.append(.package(url: "https://github.com/nerzh/telegram-vapor-bot", .upToNextMajor(from: "2.1.0")))

let package = Package(
    name: "Telegram-bot-example",
    platforms: [
        .macOS(.v12)
    ],
    dependencies: packageDependencies,
    targets: [
        .executableTarget(
            name: "Telegram-bot-example",
            dependencies: [
                .product(name: "Vapor", package: "vapor"),
                .product(name: "TelegramVaporBot", package: "telegram-vapor-bot"),
            ]
        )
    ]
)

Слайд 58 материала для разработчиков от Apple говорит нам о том, что лучше использовать декларативный синтаксис подразумевая, что вместо отрывка кода выше будет отрывок кода ниже:

// swift-tools-version:5.8

import PackageDescription

let package = Package(
    name: "Telegram-bot-example",
    platforms: [
        .macOS(.v12)
    ],
    dependencies: [
        .package(url: "https://github.com/vapor/vapor.git", .upToNextMajor(from: "4.57.0")),
        .package(url: "https://github.com/nerzh/telegram-vapor-bot", .upToNextMajor(from: "2.1.0"))
    ],
    targets: [
        .executableTarget(
            name: "Telegram-bot-example",
            dependencies: [
                .product(name: "Vapor", package: "vapor"),
                .product(name: "TelegramVaporBot", package: "telegram-vapor-bot"),
            ]
        )
    ]
)

Честно сказать, не знаю как это в теории может повлиять на что-либо, но почему нет :)

Ну и по мелочи можно убрать лишние пробельные символы после всего листинга.

nerzh commented 10 months ago

Привет, ну, эпл нам говорит пользуйтесь Xcode, а у него рейтинг 2.8