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 README.md #65647

Closed
wants to merge 2 commits into from
Closed

Update README.md #65647

wants to merge 2 commits into from

Conversation

Boby900
Copy link

@Boby900 Boby900 commented May 11, 2024

that extra period was creating confusion and I was adding two periods which was throwing an error.

Screenshot (456)

that extra period was creating confusion and I was adding two periods which was throwing an error.
@Boby900 Boby900 requested review from a team as code owners May 11, 2024 13:15
@Boby900 Boby900 requested review from ismaelrumzan and jh3y and removed request for a team May 11, 2024 13:15
@ijjk ijjk added the examples Issue/PR related to examples label May 11, 2024
@ijjk
Copy link
Member

ijjk commented May 11, 2024

Allow CI Workflow Run

  • approve CI run for commit: e051dc4

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

1 similar comment
@ijjk
Copy link
Member

ijjk commented May 11, 2024

Allow CI Workflow Run

  • approve CI run for commit: e051dc4

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

@ijjk
Copy link
Member

ijjk commented May 11, 2024

Allow CI Workflow Run

  • approve CI run for commit: d8be879

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

@samcx
Copy link
Member

samcx commented May 13, 2024

@Boby900 I think this is fine because it's outside the code snippet (there's a period after for the line below as well).

@samcx samcx closed this May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
examples Issue/PR related to examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants