Skip to content
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

feat: Misc fixes #198

Merged
merged 13 commits into from
Mar 14, 2023
Merged

feat: Misc fixes #198

merged 13 commits into from
Mar 14, 2023

Conversation

amhn
Copy link
Contributor

@amhn amhn commented Mar 12, 2023

Fixes: #180
Fixes: #187
Fixes: #188
Fixes: #189
Fixes: #190
Fixes: #191
Fixes: #192
Fixes: #193
Fixes: #194
Fixes: #195
Fixes: #196
Fixes: #197

This fixes the above issues. One issue per commit. After every commit, all tests pass.

@auto-assign auto-assign bot requested a review from smutel March 12, 2023 13:52
@amhn amhn changed the title Misc fixes feat: Misc fixes Mar 12, 2023
@amhn amhn force-pushed the misc_fixes branch 2 times, most recently from 2444a3d to 64d1428 Compare March 12, 2023 14:02
@smutel
Copy link
Owner

smutel commented Mar 13, 2023

Hello @amhn. Thanks for this PR. There is a folder called examples/exports/v3.3/v3.2. Is-it normal or it is a typo ?

@amhn
Copy link
Contributor Author

amhn commented Mar 13, 2023

That was a typo / wrong git mv call. I pushed a new verstion that fixes this.

@smutel
Copy link
Owner

smutel commented Mar 13, 2023

OK for me. I will just review the commit msg to have a pretty changelog.

@amhn
Copy link
Contributor Author

amhn commented Mar 13, 2023

Feel free to change them as needed. You should be able to push into my branch.

@smutel smutel merged commit d8c2326 into smutel:master Mar 14, 2023
@amhn amhn deleted the misc_fixes branch March 14, 2023 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment