Updated Point to be a sendable struct

[CraigSiemens]

Closes #69, closes #70

Proposed Changes

Updated Point to be a sendable struct.

Deprecated addTag(key:value:), addField(key:value:), and time(time:) in favor of using the mutable properties.

Added mutating versions of addTag(key:value:), addField(key:value:), and time(time:) for use when the result is discarded. This is to try and minimize the number of complication errors introduced with this change. The following warnings will be produced in the following use cases.

Calling init and mutating functions in the same statement

let point = InfluxDBClient.Point("p")
    .addTag(key: "t1", value: "a")
    .addTag(key: "t2", value: "b")
     `- warning: 'point.addTag(key:value:)' is deprecated: Pass tags to Point.init or use the tags property

Calling init and mutating functions in separate statements

let point = InfluxDBClient.Point("p")

point
    .addTag(key: "t1", value: "a")
     `- warning: 'change()' is deprecated: Pass tags to Point.init or use the tags property
    .addTag(key: "t2", value: "b")
     `- warning: 'point.addTag(key:value:)' is deprecated: Pass tags to Point.init or use the tags property
     `- warning: result of call to 'change()' is unused

Checklist

• CHANGELOG.md updated
• Rebased/mergeable
• A test has been added if appropriate
• swift test completes successfully
• Commit messages are conventional
• Sign CLA (if not already signed)

[bednar]

Hi @CraigSiemens, thanks for your PR.

I am fully OK with your changes. However, since this is a breaking change, please update the client version in Sources/InfluxDBSwift/InfluxDBClient.swift to 2.0.0. Additionally, please extend the description in the CHANGELOG.md with information about how the API has changed and the benefits of these changes. Once that’s done, we can proceed to quickly merge your PR.

Best Regards

[CraigSiemens]

I've added the changed requested. I also updated the minimum swift version to 5.7 since that's the version that added the Sendable protocol.

I also noticed the readme and examples are now going to be outdated. Is that something that should be updated in this PR too?

[bednar]

I've added the changed requested. I also updated the minimum swift version to 5.7 since that's the version that added the Sendable protocol.

I also noticed the readme and examples are now going to be outdated. Is that something that should be updated in this PR too?

Yes, we need to ensure the examples and README are aligned with the ‘current’ API, so please update those as well.

[bednar]

Please update README and examples.