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

Remove trailing space #8

Merged
merged 2 commits into from
Oct 22, 2024
Merged

Remove trailing space #8

merged 2 commits into from
Oct 22, 2024

Conversation

Rexios80
Copy link
Contributor

No description provided.

@gmpassos
Copy link
Owner

gmpassos commented Oct 20, 2024

Thank you for the PR.

Could you please add a unit test? Also, you need to adjust the current failing tests.

Best regards.

@Rexios80
Copy link
Contributor Author

I updated the existing tests which should sufficiently cover this change

@Rexios80
Copy link
Contributor Author

@gmpassos Do you need anything else from me on this? That CI failure does not appear to be the fault of this PR.

@gmpassos gmpassos merged commit cd8346e into gmpassos:master Oct 22, 2024
1 check failed
@Rexios80
Copy link
Contributor Author

Thank you for merging this! When do you plan on releasing it to pub.dev?

@gmpassos
Copy link
Owner

gmpassos commented Oct 24, 2024

Published (v2.0.1) 👍

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.

2 participants