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

ref(ua): rename functions #1816

Merged
merged 2 commits into from
Feb 3, 2023
Merged

ref(ua): rename functions #1816

merged 2 commits into from
Feb 3, 2023

Conversation

TBS1996
Copy link
Contributor

@TBS1996 TBS1996 commented Feb 3, 2023

This PR renames two functions:

  • RawUserAgentInfo::new() -> RawUserAgentInfo::from_headers()
  • Fix typo in samplingresult_from_rules_and_proccessing_flag.

This is a separate PR to avoid too many diffs in #1802.

#skip-changelog

@TBS1996 TBS1996 requested a review from a team February 3, 2023 14:00
@TBS1996 TBS1996 requested a review from jjbayer February 3, 2023 14:03
@jjbayer jjbayer changed the title ref: rename functions ref(ua): rename functions Feb 3, 2023
@TBS1996 TBS1996 merged commit 861966f into master Feb 3, 2023
@TBS1996 TBS1996 deleted the ref/rename_fn branch February 3, 2023 14:25
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