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.

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

Add venv creation and activation support #7

Merged
merged 28 commits into from
Feb 19, 2024
Merged

Add venv creation and activation support #7

merged 28 commits into from
Feb 19, 2024

Conversation

Aymane11
Copy link
Contributor

This PR adds the uv-venv option to create and activate a virtual environment using uv.

Usage:

steps:
  - uses: actions/checkout@v3
  - uses: actions/setup-python@v4
    with:
      python-version: "3.11"
  - uses: yezz123/setup-uv@v1
    with:
      uv-venv: "your_venv_name"
  - run: uv pip install black # this command will run in the uv environment

Solves #6

@Aymane11
Add UV venv support
2c0f2a7
@Aymane11
Add venv tests
5905e19
@Aymane11
Add venv input to actions.yml
9776a51
@Aymane11
Add list files to pipeline to check for venv creation
c098329
@Aymane11
Trigger Pipeline
e64978c
@Aymane11
Run package command
9490d31
@Aymane11
Fix venv activation command
26fe4ea
@Aymane11
Run package (again)
89ee6fe
@Aymane11
Update venv activation command (. instead of source)
20b1412
@Aymane11
Debugging to list files
5261b12
@Aymane11
Fix venv activation command
44ec2ab
@Aymane11
Update venv activation to use /bin/bash
f952efc
@Aymane11
Add check to get python path in workflow
eebdef6
@Aymane11
Setup env vars and path after venv activation
bff0a02
@Aymane11
Update python executable retrieval and add check for virtual environment
3bf2564
@Aymane11
Fix virtual environment check in default.yml
12e09ab
@Aymane11
Fix venv activation script for Windows
162abe4
@Aymane11
Update venv activation command windows
50b2b31
@Aymane11
Fix PATH for Windows
348bbab
@Aymane11
Update activateVenv function to use powershell on Windows
42c6825
@Aymane11
Update uv-venv description in action.yml
6029390
@Aymane11
Refactor venv check in pipeline
ab18458
@Aymane11
Update GitHub Actions workflows to fix deprecation warning
3e1e00e
@Aymane11
Update GitHub Actions versions
686fcdd
@Aymane11
Add uv-venv parameter to workflow
cf42ea1
@Aymane11
Remove default venv creation
fd3f928
@Aymane11
Add venv option to README
98b323b
@Aymane11
Fix README
35ee3b0
@yezz123 yezz123 linked an issue Feb 19, 2024 that may be closed by this pull request
automate the setup of the virtual environment #6
Closed
@yezz123 yezz123 self-requested a review February 19, 2024 20:30
yezz123
yezz123 approved these changes Feb 19, 2024
View reviewed changes
Copy link
Owner

@yezz123 yezz123 left a comment

Choose a reason for hiding this comment

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

Clean! 🚀

Thank you @Aymane11 🙏🏻

@yezz123 yezz123 merged commit 6fa23a8 into yezz123:main Feb 19, 2024
16 checks passed
yezz123 pushed a commit that referenced this pull request Mar 7, 2024
@Aymane11 @yezz123
Add venv creation and activation support (#7)
a7fb588
yezz123 added a commit that referenced this pull request Mar 7, 2024
@yezz123 @Aymane11 @dependabot
🐛 fix uv version implementation (#15)
4a6cab9 
* Add version support

* Add `venv` creation and activation support (#7)

* 🎨 Fix import statement in main.ts

* 📝 Remove unused scripts from package.json

* ✨ Add check-dist.yml workflow for transpiled JavaScript

* ⬆️  Bump the npm-development group with 1 update (#8)

Bumps the npm-development group with 1 update: [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node).

Updates `@types/node` from 20.11.19 to 20.11.20
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node)

---
updated-dependencies:
- dependency-name: "@types/node"
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: npm-development
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* 🔥 remove preview version

* ✅ remove preview tests

* 📝 update documentation

* ✨ Remove unused code

* 🐛 Remove lint and coverage scripts from "all" script

* ⬆️ Bump the npm-development group with 1 update (#11)

Bumps the npm-development group with 1 update: [eslint](https://github.com/eslint/eslint).


Updates `eslint` from 8.56.0 to 8.57.0
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/main/CHANGELOG.md)
- [Commits](eslint/eslint@v8.56.0...v8.57.0)

---
updated-dependencies:
- dependency-name: eslint
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: npm-development
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* ⬆️ Bump the npm-production group with 1 update (#12)

Bumps the npm-production group with 1 update: [@types/semver](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/semver).


Updates `@types/semver` from 7.5.7 to 7.5.8
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/semver)

---
updated-dependencies:
- dependency-name: "@types/semver"
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: npm-production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump the npm-development group with 2 updates (#13)

Bumps the npm-development group with 2 updates: [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) and [eslint-plugin-github](https://github.com/github/eslint-plugin-github).


Updates `@types/node` from 20.11.20 to 20.11.24
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node)

Updates `eslint-plugin-github` from 4.10.1 to 4.10.2
- [Release notes](https://github.com/github/eslint-plugin-github/releases)
- [Commits](github/eslint-plugin-github@v4.10.1...v4.10.2)

---
updated-dependencies:
- dependency-name: "@types/node"
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: npm-development
- dependency-name: eslint-plugin-github
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: npm-development
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Refactor UV installation script and add version support

* Add warning if no version provided

* Fix CI

* Fix no version case in CI

* Use uv v.0.1.12 in CI

* Update UV installer script URL

* Fix UV installation command for Windows

* Update UV installation command for Windows

* Add .ps1 extension to downloaded file

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Yasser Tahiri <yasserth19@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add support for creating and activating virtual environments

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Aymane Boumaaza <aymaneboumaaza@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yezz123 yezz123 yezz123 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

automate the setup of the virtual environment
2 participants
@Aymane11 @yezz123