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

Allow for regular strings instead of only static strings #25

Merged
merged 2 commits into from
Jul 14, 2023

Conversation

DaanDD
Copy link
Contributor

@DaanDD DaanDD commented Jul 4, 2023

Hello, thanks for creating this package!

I have made a small change, allowing for any tag string to be sent, not only static strings. Otherwise it was impossible to integrate in my code.

@johnmanjiro13
Copy link
Owner

johnmanjiro13 commented Jul 7, 2023

Hi, @DaanDD
Thank you for your contribution!

I'd like to review this pr but I have no time because I'm on vacation now.
So, please wait a few days 🙏

@johnmanjiro13
Copy link
Owner

johnmanjiro13 commented Jul 9, 2023

Could you merge main branch to yours because I fixed a test workflow?
After that, I will run tests and merge this pr.

@DaanDD
Copy link
Contributor Author

DaanDD commented Jul 13, 2023

@johnmanjiro13 Done!

Copy link
Owner

@johnmanjiro13 johnmanjiro13 left a comment

Choose a reason for hiding this comment

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

LGTM

@johnmanjiro13 johnmanjiro13 merged commit 2e22a30 into johnmanjiro13:main Jul 14, 2023
6 checks passed
@github-actions github-actions bot mentioned this pull request Jul 14, 2023
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.

None yet

2 participants