Skip to content

refactor: merge changes from master branch to v3 #135

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Mar 11, 2025

Conversation

Rolv-Apneseth
Copy link
Collaborator

Rebase was freaking out for some reason so I went with a merge. Hopefully this carries over the changes correctly.

I had to manually edit the changelog but let me know if it's wrong. Do you use something for generating it? I saw git-cliff listed at the bottom but for me the output is different.

okaerin and others added 4 commits February 21, 2025 12:29
…cept: application/json` header (jeertmans#129)

* fix query to form encoded payload

* change all requests to form encoded

* add accept header "application/json"

* revert /words to query encoded

* update changelog
* rename unrelease

* prepare next unreleased section

* add release process and changelog update

* add cargo release install cmd

* omit workspace
* fix doc

* update changelog

* apply review comments

- clarified purpose of cargo release
- simplified release preparation (removed parameters are in Cargo.toml)

* Update RELEASE-PROCESS.md

---------

Co-authored-by: Jérome Eertmans <jeertmans@icloud.com>
Copy link

codspeed-hq bot commented Mar 10, 2025

CodSpeed Performance Report

Merging #135 will not alter performance

Comparing Rolv-Apneseth:rebase-v3 (1a1aa4c) with v3 (cf9462b)

Summary

✅ 6 untouched benchmarks

@Rolv-Apneseth
Copy link
Collaborator Author

Had to fix an import too. Otherwise seems OK.

@jeertmans we should probably merge this one before #134, when you get the chance to review

Copy link
Owner

@jeertmans jeertmans left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks @Rolv-Apneseth! Don't know why the date was removed in main, probably missed it.

@jeertmans jeertmans merged commit 4bb3d90 into jeertmans:v3 Mar 11, 2025
8 of 11 checks passed
@Rolv-Apneseth Rolv-Apneseth deleted the rebase-v3 branch March 11, 2025 13:09
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.

3 participants