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

[CCI][GUIDE] Minor fixes to poetry docs #494

Merged
merged 4 commits into from
Sep 15, 2023

Conversation

roma2023
Copy link
Contributor

Description

Added python to run command of poetry. Now it runs as poetry run python hello/hello.py .

Issues Resolved

Issue #493 resolved.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Raman Saparkhan <[email protected]>
@codecov
Copy link

codecov bot commented Sep 14, 2023

Codecov Report

Merging #494 (dcea653) into main (c8b04a5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #494   +/-   ##
=======================================
  Coverage   70.92%   70.92%           
=======================================
  Files          81       81           
  Lines        7732     7732           
=======================================
  Hits         5484     5484           
  Misses       2248     2248           

@saimedhi saimedhi added the skip-changelog Skips changelog verifier label Sep 14, 2023
saimedhi
saimedhi previously approved these changes Sep 14, 2023
Signed-off-by: Raman Saparkhan <[email protected]>
@saimedhi
Copy link
Collaborator

saimedhi commented Sep 14, 2023

@roma2023, Issue is not specific to windows. I experienced it in Mac when running security/roles or security/users samples.

@roma2023
Copy link
Contributor Author

@saimedhi Then, I believe the issue was more about the run command rather than the machine it was being run.

Signed-off-by: Raman Saparkhan <[email protected]>
@saimedhi saimedhi marked this pull request as draft September 14, 2023 19:23
@saimedhi
Copy link
Collaborator

Change it to ready for review after you are done updating. Thank you :)

Signed-off-by: Raman Saparkhan <[email protected]>
@roma2023 roma2023 marked this pull request as ready for review September 14, 2023 19:45
@roma2023 roma2023 requested a review from saimedhi September 14, 2023 20:03
@dblock dblock merged commit 29c6445 into opensearch-project:main Sep 15, 2023
52 checks passed
roma2023 added a commit to roma2023/opensearch-py that referenced this pull request Dec 28, 2023
* minor fixes to poetry docs

Signed-off-by: Raman Saparkhan <[email protected]>

* updated CHANGELOG.md

Signed-off-by: Raman Saparkhan <[email protected]>

* updated CHANGELOG.md

Signed-off-by: Raman Saparkhan <[email protected]>

* fixed CHANGELOG.md

Signed-off-by: Raman Saparkhan <[email protected]>

---------

Signed-off-by: Raman Saparkhan <[email protected]>
Signed-off-by: roma2023 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog Skips changelog verifier
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants