Skip to content

Commit

Permalink
Fix string truncation (#36)
Browse files Browse the repository at this point in the history
* Fix string truncation

* Update CI

* Update deny

* Add single join job

* Update CHANGELOG
  • Loading branch information
Jake-Shadle authored Mar 5, 2024
1 parent 7e0eca4 commit a2eb931
Show file tree
Hide file tree
Showing 6 changed files with 1,252 additions and 25 deletions.
16 changes: 11 additions & 5 deletions .github/workflows/rust-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ jobs:
name: Lint
runs-on: ubuntu-22.04
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@stable
with:
components: "clippy, rustfmt"
Expand All @@ -35,10 +35,10 @@ jobs:
name: Test
strategy:
matrix:
os: [ubuntu-22.04, windows-2022, macOS-12]
os: [ubuntu-22.04, windows-2022, macOS-14]
runs-on: ${{ matrix.os }}
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@stable
- uses: Swatinem/rust-cache@v2
- run: cargo fetch
Expand All @@ -51,16 +51,22 @@ jobs:
name: cargo-deny
runs-on: ubuntu-22.04
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: EmbarkStudios/cargo-deny-action@v1

publish-check:
name: Publish Check
runs-on: ubuntu-22.04
steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@stable
- uses: Swatinem/rust-cache@v2
- run: cargo fetch
- name: cargo publish check
run: cargo publish --dry-run --manifest-path sdk/Cargo.toml

test_success:
runs-on: ubuntu-22.04
needs: [lint,test,publish-check,deny-check]
steps:
- run: echo "All test jobs passed"
5 changes: 4 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,13 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

<!-- markdownlint-disable blanks-around-headers no-duplicate-header blanks-around-lists -->
<!-- markdownlint-disable blanks-around-headings no-duplicate-heading blanks-around-lists -->

<!-- next-header -->
## [Unreleased] - ReleaseDate
### Fixed
- [PR#36](https://github.com/EmbarkStudios/discord-sdk/pull/36) fixed [#35](https://github.com/EmbarkStudios/discord-sdk/issues/35) by properly truncating utf-8 strings.

## [0.3.5] - 2023-10-04
### Changed
- [PR#34](https://github.com/EmbarkStudios/discord-sdk/pull/34) replaced `base64` with `data-encoding`.
Expand Down
Loading

0 comments on commit a2eb931

Please sign in to comment.