Skip to content

Conversation

@github-actions
Copy link
Contributor

This PR was automatically opened by a GitHub action. Review the changes included in this PR and determine if they should be included in the release branch. If yes, merge the PR. Otherwise revert changes that should not be included on this branch.

MojtabaHs and others added 3 commits November 11, 2025 08:39
* fix: discontiguous attributed string

* fix: not divisable

* fix: item delimiter

* fix: contiguous sequence
Reverting this from main for now to figure out test breakages
…naName:)`. (#1286)

* Import implementation for String Encoding Names from other repo.

- source: https://github.com/YOCKOW/SF-StringEncodingNameImpl

* Import tests for String Encoding Names from other repo.

- source: https://github.com/YOCKOW/SF-StringEncodingNameImpl/blob/0.4.0/Tests/StringEncodingNameImplTests/StringEncodingNameParserTests.swift

* Remove dead code in terms of the current proposal.

* Use `Testing` for String Encoding Names tests.

* NFC: Fix indentation in "String+Encoding+Names.swift".

* SF-0033: Adjust comments/attributes to match the accepted proposal.

* Auto-generate Swift source code for IANA Charset names.

* Remove unnecessary `@inlinable`.

* Simplify `String.init(ianaName:)`.

* Add new files related to SF-0033 to CMakeLists.txt.

* Rewrite script in Swift instead of Python.

In response to: #1286 (comment)

* Simplify logic to parse IANA Charset names.

In response to:
- #1286 (comment)
- #1286 (comment)
- #1286 (comment)

* Fix spelling of functions for "case-insensitively".

In response to:
- #1286 (comment)

* Remove redundant nested function in `String.Encoding(ianaName:)`.

In response to:
- #1286 (comment)
@jmschonfeld jmschonfeld reopened this Nov 12, 2025
@jmschonfeld jmschonfeld marked this pull request as ready for review November 12, 2025 19:46
@jmschonfeld jmschonfeld merged commit 4dcbbcc into future Nov 12, 2025
37 of 38 checks passed
@jmschonfeld jmschonfeld deleted the automerge/merge-main-2025-11-12_09-05 branch November 12, 2025 19:46
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.

5 participants