Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: apple/swift-openapi-generator
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 1.6.0
Choose a base ref
...
head repository: apple/swift-openapi-generator
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 1.7.0
Choose a head ref
  • 5 commits
  • 98 files changed
  • 2 contributors

Commits on Dec 19, 2024

  1. [SOAR-0013] Idiomatic naming strategy (#683)

    ### Motivation
    
    See proposal
    
    ### Modifications
    
    N/A
    
    ### Result
    
    N/A
    
    ### Test Plan
    
    N/A
    czechboy0 authored Dec 19, 2024
    Configuration menu
    Copy the full SHA
    dcfb1ed View commit details
    Browse the repository at this point in the history
  2. Update Examples and IntegrationTest to use idiomatic naming (#706)

    ### Motivation
    
    Since idiomatic naming is our new recommended setting, use it in
    Examples and IntegrationTest.
    
    ### Modifications
    
    Added idiomatic naming to all examples and made any necessary changes.
    Plus a few modernizations, like moving to Hummingbird 2.0 and to
    Prometheus 2.0.
    
    ### Result
    
    More up-to-date examples, easier to copy out by adopters, they now get
    our recommended starting settings.
    
    ### Test Plan
    
    Ran and tested locally.
    
    ---------
    
    Co-authored-by: Si Beaumont <simonjbeaumont@gmail.com>
    czechboy0 and simonjbeaumont authored Dec 19, 2024
    Configuration menu
    Copy the full SHA
    9c01b32 View commit details
    Browse the repository at this point in the history

Commits on Jan 6, 2025

  1. Add a CLI option for selecting the naming strategy (#708)

    ### Motivation
    
    The recently introduced naming strategy is only configurable in the
    config file, but pure CLI users might also want to use it without
    creating a config file.
    
    ### Modifications
    
    Add a `--naming-strategy` option to the `swift-openapi-generator` CLI.
    
    ### Result
    
    Users can use the new naming strategy without creating a config file, if
    they didn't have one before.
    
    ### Test Plan
    
    Tested manually on a sample project.
    czechboy0 authored Jan 6, 2025
    Configuration menu
    Copy the full SHA
    6593b1b View commit details
    Browse the repository at this point in the history
  2. Add missing imports to fix Swift 6.1+ (#709)

    ### Motivation
    
    Currently unit tests on Swift 6.1+ toolchains are failing due to a few
    missing imports.
    
    That these are coming up is good, that's why we've enabled
    `MemberImportVisibility`.
    
    ### Modifications
    
    Add the missing imports to stop relying on implicitly imported extension
    methods.
    
    ### Result
    
    Unit tests compile again on Swift 6.1+.
    
    ### Test Plan
    
    Tested locally, will further verify CI is happy.
    czechboy0 authored Jan 6, 2025
    Configuration menu
    Copy the full SHA
    f3c32cc View commit details
    Browse the repository at this point in the history
  3. [Cleanup] Make Config's namingStrategy and accessModifier params cons…

    …istent (#713)
    
    ### Motivation
    
    A missed fixup from #708:
    #708 (comment)
    
    ### Modifications
    
    Removed the default from Config.init and forced all callers to make a
    choice on the naming strategy, just like for accessModifier.
    
    ### Result
    
    Consistent handling of these two parameters.
    
    ### Test Plan
    
    Tests passed locally.
    czechboy0 authored Jan 6, 2025
    Configuration menu
    Copy the full SHA
    84b693f View commit details
    Browse the repository at this point in the history
Loading