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

Migrate to precommit.ci #487

Merged
merged 1 commit into from Jan 17, 2022
Merged

Conversation

BeyondEvil
Copy link
Contributor

No description provided.

@BeyondEvil BeyondEvil requested a review from a team January 17, 2022 19:09
Copy link
Member

@nicoddemus nicoddemus left a comment

Choose a reason for hiding this comment

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

I see there's still a linting job running in there though.

@BeyondEvil
Copy link
Contributor Author

I see there's still a linting job running in there though.

Yes, that's because the action uses an absolute path to the tests.yml and not the one in the PR.

As far as I can tell, unfortunately, you can't use a relative path.

@BeyondEvil BeyondEvil merged commit fdcd5c6 into pytest-dev:master Jan 17, 2022
@BeyondEvil BeyondEvil deleted the migrate-pre-commit branch January 17, 2022 20:14
BeyondEvil added a commit that referenced this pull request Jan 26, 2022
* remove phantomjs dependency (#424)

* properly classify all npm dependencies (#425)

* Move the changelog to read the docs (#423)

* split plugin.py into smaller files (#427)

* Implement the visible URL query parameter to control visibility of test results on page load. (#433)

* enable control of test result visability via query params

* Allow for redacting of environment table values (#431)

* Disable Codecov (#480)

* Disable Codecov

* Disable pypy3 on mac

* Add Tests.yml reusable workflow (#484)

* Use the tests reusable workflow (#486)

* Migrate to precommit.ci (#487)

* Separate Nightly workflow (#488)

Co-authored-by: Gleb Nikonorov <gleb.i.nikonorov@gmail.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@ssbarnea
Copy link
Member

ssbarnea commented Jul 4, 2022

@BeyondEvil While I use pre-commit.ci myself almost everywhere, I never rely on it for CI pipelines and I do have separated GHA lint pipeline that runs linting using pre-commit. That is for two reasons:

  • Safety: you do not want to add another point of failure in testing (another SaaS)
  • pre-commit.ci is unable to run all types of hooks, any hook that needs network access will not be usable, like docker based ones or ones that check dependencies. In this project we happen to not have such hooks (all are compatible) but still.

I mentioned that as it might prove useful.

@BeyondEvil
Copy link
Contributor Author

I see. All valid points. But at this time I feel that if it’s good enough for the main pytest project it’s good enough for us.

With limited time, at least I have more pressing issues to deal with.

Speaking of, any thoughts on why the windows tests started failing?

@BeyondEvil
Copy link
Contributor Author

FYI, you can now use relative paths. 😅

@ssbarnea
Copy link
Member

ssbarnea commented Jul 4, 2022

Sorry but Windows is the only platform about which I do not care, so I did not look at it. I already wasted almost two weeks making Ansible work under GHA WSL1, ended up with an instant heachache when I see Windows ;)

@BeyondEvil
Copy link
Contributor Author

Yeah, I hear ya.

I have a strong urge to remove official support for windows.

If it works it works, and I can accept patches. But that’s about it.

I don’t own a PC and I have a M1 with no feasible means of running a VM yet.

@ssbarnea
Copy link
Member

ssbarnea commented Jul 4, 2022

Not that I expect you do work on it, but I have an M1 too and I installed Windows 11 arm64 edition under Parallels and works quite nice and FAST!. Just do not expect to be able to use podman/docker/wsl inside that VM because M1 does not allow nested-virtualization. In fact I even share my code repository across those systems, so I do not need to clone/sync code when testing under a different platform. I was planning to write a blog post about that, but time....

@BeyondEvil
Copy link
Contributor Author

Yeah, but parallels isn't free. And I don't have a need for it for work and I don't feel like paying for something w.r.t. unpaid OS work.

I'll just wait until the folks at VirtualBox catches up.

Still nice, will read that blog-post.

@ssbarnea
Copy link
Member

ssbarnea commented Jul 4, 2022

That will not catch-up that time, they announced that there are no plans for arm64 on virtualbox. Also UTM seems might be something you want to investigate as open-source (its a gui on top of qemu)

@BeyondEvil
Copy link
Contributor Author

Yeah, I tried UTM but couldn't get it to work. I might revisit.

Yeah, I know what they announced. But unless they want to become irrelevant they'll have to as Apple will move "everything" to ARM.

BeyondEvil added a commit to BeyondEvil/pytest-html that referenced this pull request Apr 3, 2023
* remove phantomjs dependency (pytest-dev#424)

* properly classify all npm dependencies (pytest-dev#425)

* Move the changelog to read the docs (pytest-dev#423)

* split plugin.py into smaller files (pytest-dev#427)

* Implement the visible URL query parameter to control visibility of test results on page load. (pytest-dev#433)

* enable control of test result visability via query params

* Allow for redacting of environment table values (pytest-dev#431)

* Disable Codecov (pytest-dev#480)

* Disable Codecov

* Disable pypy3 on mac

* Add Tests.yml reusable workflow (pytest-dev#484)

* Use the tests reusable workflow (pytest-dev#486)

* Migrate to precommit.ci (pytest-dev#487)

* Separate Nightly workflow (pytest-dev#488)

Co-authored-by: Gleb Nikonorov <gleb.i.nikonorov@gmail.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants