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.

Sign up for GitHub

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

Jump to bottom

Explicit type conversion #335

Closed
marco-prontera opened this issue Feb 16, 2022 · 0 comments · Fixed by #336
Closed

Explicit type conversion #335

marco-prontera opened this issue Feb 16, 2022 · 0 comments · Fixed by #336
Labels
enhancement New feature or request

Comments

@marco-prontera
Copy link
Contributor

Hi guys, reading the code a bit I noticed there are a lot of implicit string conversions. I'll open a PR soon to change these to a more explicit version, it should be more readable later.
@marco-prontera marco-prontera added the enhancement New feature or request label Feb 16, 2022
@marco-prontera marco-prontera mentioned this issue Feb 16, 2022
Merged
kenario added a commit that referenced this issue Apr 4, 2022
@kenario
Merge pull request #336 from Marco-Prontera/feature/335
f3df257 
[#335] Explicit type conversion
sevriugin pushed a commit to didomi/iabtcf-es-archived that referenced this issue Aug 29, 2022
@kenario @sevriugin
Merge pull request InteractiveAdvertisingBureau#336 from Marco-Pronte…
e870d24 
…ra/feature/335

[InteractiveAdvertisingBureau#335] Explicit type conversion
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[#335] Explicit type conversion marco-prontera/iabtcf-es
1 participant
@marco-prontera