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

Update AWS_APIfromTor.yaml #11765

Closed
wants to merge 1 commit into from

Conversation

LorenzoCristini
Copy link

@LorenzoCristini LorenzoCristini commented Feb 5, 2025

Replace "contains" with "has" statement

Required items, please complete

Change(s):

  • Replaced "contains" statement with "has" statement

Reason for Change(s):

  • This should enhance overall performance due to tokenization of the has statement

Version Updated:

  • Yes

Testing Completed:

  • Yes

Checked that the validations are passing and have addressed any issues that are present:

  • Yes

Replace "contains" with "has" statement
@LorenzoCristini LorenzoCristini requested review from a team as code owners February 5, 2025 13:49
@v-prasadboke v-prasadboke self-assigned this Feb 6, 2025
@v-prasadboke
Copy link
Contributor

@v-prasadboke
Copy link
Contributor

Please proceed with the above requested change

@LorenzoCristini
Copy link
Author

Hi @v-prasadboke , I'm not currently able to proceed with the package . I will close this request and will open a bigger pull request in the future to add the statements in other analytics.
Thanks.

@LorenzoCristini LorenzoCristini deleted the patch-1 branch February 17, 2025 10:50
@LorenzoCristini LorenzoCristini restored the patch-1 branch February 17, 2025 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants