Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Updated README.md in setup-python #472

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
Merged
merged 27 commits into from
Aug 2, 2022
Merged

Updated README.md in setup-python #472

merged 27 commits into from
Aug 2, 2022

Conversation

IvanZosimov
Copy link
Contributor

Description:
In the scope of this PR the README.md file was updated to be more readable and understandable. advanced-usage.md file was created.

Related issue:
https://github.com/actions/virtual-environments-internal/issues/3964

IvanZosimov added 22 commits July 12, 2022 16:38
Rearrange README.md, add advanced-usage.md
9a40041
Change tool cache wording
6dd8ff7
Update REAMDE.md and advanced-usage.md
746f28a
Update action.yml file
09086cc
Fix review points
61fb4e4
Fix review points
386e4ea
Fix action.yml file
799afeb
Fix documentation
5517d5f 
Docs were updated to incorporate changes regarding tool cache folder
on the self-hosted runner and changes in resolveVersionInput()
Change part with realted to self-hosted runners
00d9c42
Fix typo
889226a
Fix typo
0d94a5d
Change yml and rebuild action
b2241a4
Sync with Main branch
1f0a39a
Fix merge artifacts
dd40245
Fix grammar in both documents
a624f1f
Fix typo in advanced-usage.md
b152b04
Fix review points
81cda82
Fix typo
c6e66a7
Change contents to make them more readable
fd6f59d
Change advanced-usage.md
853c012
Revert changes
fe396d3
Fix typo
72394d1
@IvanZosimov IvanZosimov requested a review from a team July 26, 2022 15:36
brcrista
brcrista reviewed Jul 26, 2022
View reviewed changes
brcrista
brcrista reviewed Jul 26, 2022
View reviewed changes
brcrista
brcrista reviewed Jul 26, 2022
View reviewed changes
brcrista
brcrista reviewed Jul 26, 2022
View reviewed changes
brcrista
brcrista reviewed Jul 26, 2022
View reviewed changes
brcrista
brcrista reviewed Jul 26, 2022
View reviewed changes
@IvanZosimov IvanZosimov requested a review from brcrista July 28, 2022 07:28
brcrista
brcrista reviewed Aug 1, 2022
View reviewed changes
brcrista
brcrista reviewed Aug 1, 2022
View reviewed changes
brcrista
brcrista reviewed Aug 1, 2022
View reviewed changes
brcrista
brcrista reviewed Aug 1, 2022
View reviewed changes
brcrista
brcrista reviewed Aug 1, 2022
View reviewed changes
brcrista
brcrista reviewed Aug 1, 2022
View reviewed changes
brcrista
brcrista approved these changes Aug 1, 2022
View reviewed changes
Copy link
Contributor

@brcrista brcrista left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Big improvement. Thanks for doing this!

@marko-zivic-93 marko-zivic-93 merged commit b55428b into actions:main Aug 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
4 more reviewers

@vsafonkin vsafonkin vsafonkin approved these changes

@dmitry-shibanov dmitry-shibanov dmitry-shibanov approved these changes

@brcrista brcrista brcrista approved these changes

@marko-zivic-93 marko-zivic-93 marko-zivic-93 approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@IvanZosimov @vsafonkin @dmitry-shibanov @brcrista @marko-zivic-93