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

Speed up Hatch installation #282

Merged

gaborbernat merged 1 commit into main from hatch-action

May 22, 2024

Collaborator

ofek commented May 22, 2024

This is the new way, speedup on Windows ~35-50%, other platforms 2-3x


          Speed up Hatch installation

92472b5

ofek requested review from gaborbernat, Julian and RonnyPfannschmidt as code owners

May 22, 2024 21:47

gaborbernat approved these changes

View reviewed changes

Member

gaborbernat commented May 22, 2024

It seems that it is not working.

Collaborator Author

ofek commented May 22, 2024

Yeah, figuring out the best way to fix Windows. I can't just do ./main.sh because that is in the directory where the action is checked out so you must do a full path. When the shell is bash the backslashes aren't treated literally and therefore get stripped which makes it so the full path doesn't exist. When the shell is not specified it works because then Windows uses PowerShell but every step of an action requires an explicit shell... so might have to have 2 steps that are conditional based on the platform because you cannot use conditional syntax within the shell value. Nightmare.

Collaborator Author

ofek commented May 22, 2024

Windows:
- Old: 35s
- New: 12s
Linux:
- Old: 12s
- New: 2s
macOS:
- Old: 10s
- New: 5s, although on macOS 14/ARM it's similar to Linux (sometimes 3 seconds rather than 2, Linux is consistently 2)

Collaborator Author

ofek commented May 22, 2024

Everything is fixed now, thanks for your patience!

gaborbernat approved these changes

View reviewed changes

gaborbernat merged commit 12cc712 into main

29 checks passed

gaborbernat deleted the hatch-action branch

May 22, 2024 22:50

ldetmer referenced this pull request in googleapis/sdk-platform-java


          deps: update dependency platformdirs to v4.3.2 (#3176)

4f2f9e0

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [platformdirs](https://redirect.github.com/platformdirs/platformdirs)
| `==4.2.2` -> `==4.3.2` |
[![age](https://developer.mend.io/api/mc/badges/age/pypi/platformdirs/4.3.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/pypi/platformdirs/4.3.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/pypi/platformdirs/4.2.2/4.3.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/pypi/platformdirs/4.2.2/4.3.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>platformdirs/platformdirs (platformdirs)</summary>

###
[`v4.3.2`](https://redirect.github.com/tox-dev/platformdirs/releases/tag/4.3.2)

[Compare
Source](https://redirect.github.com/platformdirs/platformdirs/compare/4.3.1...4.3.2)

<!-- Release notes generated using configuration in .github/release.yml
at main -->

#### What's Changed

- Use uv as installer by
[@&#8203;gaborbernat](https://redirect.github.com/gaborbernat) in
[https://github.com/tox-dev/platformdirs/pull/300](https://redirect.github.com/tox-dev/platformdirs/pull/300)
- Fix multi-path returned from `_path` methods on MacOS by
[@&#8203;matthewhughes934](https://redirect.github.com/matthewhughes934)
in
[https://github.com/tox-dev/platformdirs/pull/299](https://redirect.github.com/tox-dev/platformdirs/pull/299)

#### New Contributors

-
[@&#8203;matthewhughes934](https://redirect.github.com/matthewhughes934)
made their first contribution in
[https://github.com/tox-dev/platformdirs/pull/299](https://redirect.github.com/tox-dev/platformdirs/pull/299)

**Full Changelog**:
tox-dev/platformdirs@4.3.1...4.3.2

###
[`v4.3.1`](https://redirect.github.com/tox-dev/platformdirs/releases/tag/4.3.1)

[Compare
Source](https://redirect.github.com/platformdirs/platformdirs/compare/4.3.0...4.3.1)

<!-- Release notes generated using configuration in .github/release.yml
at main -->

**Full Changelog**:
tox-dev/platformdirs@4.3.0...4.3.1

###
[`v4.3.0`](https://redirect.github.com/tox-dev/platformdirs/releases/tag/4.3.0)

[Compare
Source](https://redirect.github.com/platformdirs/platformdirs/compare/4.2.2...4.3.0)

<!-- Release notes generated using configuration in .github/release.yml
at main -->

#### What's Changed

- Speed up Hatch installation by
[@&#8203;ofek](https://redirect.github.com/ofek) in
[https://github.com/tox-dev/platformdirs/pull/282](https://redirect.github.com/tox-dev/platformdirs/pull/282)
- Test with Python 3.13 by
[@&#8203;edgarrmondragon](https://redirect.github.com/edgarrmondragon)
in
[https://github.com/tox-dev/platformdirs/pull/289](https://redirect.github.com/tox-dev/platformdirs/pull/289)
- Test with latest PyPy by
[@&#8203;edgarrmondragon](https://redirect.github.com/edgarrmondragon)
in
[https://github.com/tox-dev/platformdirs/pull/290](https://redirect.github.com/tox-dev/platformdirs/pull/290)
- Use `include-hidden-files: true` to upload coverage artifacts by
[@&#8203;edgarrmondragon](https://redirect.github.com/edgarrmondragon)
in
[https://github.com/tox-dev/platformdirs/pull/298](https://redirect.github.com/tox-dev/platformdirs/pull/298)
- Ensure PlatformDirs is valid superclass type for mypy AND not an
abstract class for other checkers by
[@&#8203;Avasam](https://redirect.github.com/Avasam) in
[https://github.com/tox-dev/platformdirs/pull/295](https://redirect.github.com/tox-dev/platformdirs/pull/295)

#### New Contributors

- [@&#8203;edgarrmondragon](https://redirect.github.com/edgarrmondragon)
made their first contribution in
[https://github.com/tox-dev/platformdirs/pull/289](https://redirect.github.com/tox-dev/platformdirs/pull/289)
- [@&#8203;Avasam](https://redirect.github.com/Avasam) made their first
contribution in
[https://github.com/tox-dev/platformdirs/pull/295](https://redirect.github.com/tox-dev/platformdirs/pull/295)

**Full Changelog**:
tox-dev/platformdirs@4.2.2...4.3.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR was generated by [Mend Renovate](https://mend.io/renovate/).
View the [repository job
log](https://developer.mend.io/github/googleapis/sdk-platform-java).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC41OS4yIiwidXBkYXRlZEluVmVyIjoiMzguNTkuMiIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOltdfQ==-->

ldetmer referenced this pull request in googleapis/sdk-platform-java


          deps: update dependency platformdirs to v4.3.2 (#3176)

0327f1c

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [platformdirs](https://redirect.github.com/platformdirs/platformdirs)
| `==4.2.2` -> `==4.3.2` |
[![age](https://developer.mend.io/api/mc/badges/age/pypi/platformdirs/4.3.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/pypi/platformdirs/4.3.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/pypi/platformdirs/4.2.2/4.3.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/pypi/platformdirs/4.2.2/4.3.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>platformdirs/platformdirs (platformdirs)</summary>

###
[`v4.3.2`](https://redirect.github.com/tox-dev/platformdirs/releases/tag/4.3.2)

[Compare
Source](https://redirect.github.com/platformdirs/platformdirs/compare/4.3.1...4.3.2)

<!-- Release notes generated using configuration in .github/release.yml
at main -->

#### What's Changed

- Use uv as installer by
[@&#8203;gaborbernat](https://redirect.github.com/gaborbernat) in
[https://github.com/tox-dev/platformdirs/pull/300](https://redirect.github.com/tox-dev/platformdirs/pull/300)
- Fix multi-path returned from `_path` methods on MacOS by
[@&#8203;matthewhughes934](https://redirect.github.com/matthewhughes934)
in
[https://github.com/tox-dev/platformdirs/pull/299](https://redirect.github.com/tox-dev/platformdirs/pull/299)

#### New Contributors

-
[@&#8203;matthewhughes934](https://redirect.github.com/matthewhughes934)
made their first contribution in
[https://github.com/tox-dev/platformdirs/pull/299](https://redirect.github.com/tox-dev/platformdirs/pull/299)

**Full Changelog**:
tox-dev/platformdirs@4.3.1...4.3.2

###
[`v4.3.1`](https://redirect.github.com/tox-dev/platformdirs/releases/tag/4.3.1)

[Compare
Source](https://redirect.github.com/platformdirs/platformdirs/compare/4.3.0...4.3.1)

<!-- Release notes generated using configuration in .github/release.yml
at main -->

**Full Changelog**:
tox-dev/platformdirs@4.3.0...4.3.1

###
[`v4.3.0`](https://redirect.github.com/tox-dev/platformdirs/releases/tag/4.3.0)

[Compare
Source](https://redirect.github.com/platformdirs/platformdirs/compare/4.2.2...4.3.0)

<!-- Release notes generated using configuration in .github/release.yml
at main -->

#### What's Changed

- Speed up Hatch installation by
[@&#8203;ofek](https://redirect.github.com/ofek) in
[https://github.com/tox-dev/platformdirs/pull/282](https://redirect.github.com/tox-dev/platformdirs/pull/282)
- Test with Python 3.13 by
[@&#8203;edgarrmondragon](https://redirect.github.com/edgarrmondragon)
in
[https://github.com/tox-dev/platformdirs/pull/289](https://redirect.github.com/tox-dev/platformdirs/pull/289)
- Test with latest PyPy by
[@&#8203;edgarrmondragon](https://redirect.github.com/edgarrmondragon)
in
[https://github.com/tox-dev/platformdirs/pull/290](https://redirect.github.com/tox-dev/platformdirs/pull/290)
- Use `include-hidden-files: true` to upload coverage artifacts by
[@&#8203;edgarrmondragon](https://redirect.github.com/edgarrmondragon)
in
[https://github.com/tox-dev/platformdirs/pull/298](https://redirect.github.com/tox-dev/platformdirs/pull/298)
- Ensure PlatformDirs is valid superclass type for mypy AND not an
abstract class for other checkers by
[@&#8203;Avasam](https://redirect.github.com/Avasam) in
[https://github.com/tox-dev/platformdirs/pull/295](https://redirect.github.com/tox-dev/platformdirs/pull/295)

#### New Contributors

- [@&#8203;edgarrmondragon](https://redirect.github.com/edgarrmondragon)
made their first contribution in
[https://github.com/tox-dev/platformdirs/pull/289](https://redirect.github.com/tox-dev/platformdirs/pull/289)
- [@&#8203;Avasam](https://redirect.github.com/Avasam) made their first
contribution in
[https://github.com/tox-dev/platformdirs/pull/295](https://redirect.github.com/tox-dev/platformdirs/pull/295)

**Full Changelog**:
tox-dev/platformdirs@4.2.2...4.3.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR was generated by [Mend Renovate](https://mend.io/renovate/).
View the [repository job
log](https://developer.mend.io/github/googleapis/sdk-platform-java).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC41OS4yIiwidXBkYXRlZEluVmVyIjoiMzguNTkuMiIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOltdfQ==-->

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet