Skip to content

Commit

Permalink
build: Use node 22 (#262)
Browse files Browse the repository at this point in the history
* build: Use node 22

* revisit docs

* use python 3.12

* update README

* bump cucumber reporter version

* json report file config now needs quotes

* do not use npm cache for cucumber test

* test cucumber test

* restore test

* downgrade to @cucumber/cucumber@v9

* add cached npm config back
  • Loading branch information
tianfeng92 committed Jun 11, 2024
1 parent a9f44fa commit 1416420
Show file tree
Hide file tree
Showing 19 changed files with 3,030 additions and 2,227 deletions.
1 change: 0 additions & 1 deletion .eslintignore

This file was deleted.

23 changes: 0 additions & 23 deletions .eslintrc

This file was deleted.

36 changes: 18 additions & 18 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,13 +4,13 @@ on:
workflow_dispatch:
inputs:
releaseType:
description: 'Release type - major, minor or patch'
description: "Release type - major, minor or patch"
required: false
default: ''
default: ""
preReleaseFlavor:
description: 'Pre-Release flavor - rc, beta, or anything'
description: "Pre-Release flavor - rc, beta, or anything"
required: false
default: ''
default: ""

jobs:
create-release-draft:
Expand Down Expand Up @@ -44,8 +44,8 @@ jobs:
if: ${{ steps.prep.outputs.tag_name == '' }}
uses: actions/setup-node@v4
with:
node-version-file: '.nvmrc'
cache: npm
node-version-file: ".nvmrc"
cache: "npm"

- name: Setup Git
if: ${{ steps.prep.outputs.tag_name == '' }}
Expand Down Expand Up @@ -131,8 +131,8 @@ jobs:
if: ${{ steps.prep.outputs.asset_id == '' }}
uses: actions/setup-node@v4
with:
node-version-file: '.nvmrc'
cache: npm
node-version-file: ".nvmrc"
cache: "npm"

- name: Update Release Version
if: ${{ steps.prep.outputs.asset_id == '' }}
Expand Down Expand Up @@ -204,8 +204,8 @@ jobs:
if: ${{ steps.prep.outputs.asset_id == '' }}
uses: actions/setup-node@v4
with:
node-version-file: '.nvmrc'
cache: npm
node-version-file: ".nvmrc"
cache: "npm"

- name: Update Release Version
if: ${{ steps.prep.outputs.asset_id == '' }}
Expand Down Expand Up @@ -259,8 +259,8 @@ jobs:
runs-on: ubuntu-latest
needs: publish-release
env:
SAUCE_ACCESS_KEY: ${{secrets.SAUCE_ACCESS_KEY}}
SAUCE_USERNAME: ${{secrets.SAUCE_USERNAME}}
SAUCE_ACCESS_KEY: ${{ secrets.SAUCE_ACCESS_KEY }}
SAUCE_USERNAME: ${{ secrets.SAUCE_USERNAME }}
steps:
- name: Checkout
uses: actions/checkout@v4
Expand Down Expand Up @@ -292,8 +292,8 @@ jobs:
runs-on: ubuntu-latest
needs: publish-release
env:
SAUCE_ACCESS_KEY: ${{secrets.SAUCE_ACCESS_KEY}}
SAUCE_USERNAME: ${{secrets.SAUCE_USERNAME}}
SAUCE_ACCESS_KEY: ${{ secrets.SAUCE_ACCESS_KEY }}
SAUCE_USERNAME: ${{ secrets.SAUCE_USERNAME }}
steps:
- name: Checkout
uses: actions/checkout@v4
Expand Down Expand Up @@ -325,8 +325,8 @@ jobs:
runs-on: ubuntu-latest
needs: publish-release
env:
SAUCE_ACCESS_KEY: ${{secrets.SAUCE_ACCESS_KEY}}
SAUCE_USERNAME: ${{secrets.SAUCE_USERNAME}}
SAUCE_ACCESS_KEY: ${{ secrets.SAUCE_ACCESS_KEY }}
SAUCE_USERNAME: ${{ secrets.SAUCE_USERNAME }}
steps:
- name: Checkout
uses: actions/checkout@v4
Expand Down Expand Up @@ -359,8 +359,8 @@ jobs:
runs-on: ubuntu-latest
needs: publish-release
env:
SAUCE_ACCESS_KEY: ${{secrets.SAUCE_ACCESS_KEY}}
SAUCE_USERNAME: ${{secrets.SAUCE_USERNAME}}
SAUCE_ACCESS_KEY: ${{ secrets.SAUCE_ACCESS_KEY }}
SAUCE_USERNAME: ${{ secrets.SAUCE_USERNAME }}
steps:
- name: Checkout
uses: actions/checkout@v4
Expand Down
40 changes: 20 additions & 20 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@ on:
- main

env:
SAUCE_ACCESS_KEY: ${{secrets.SAUCE_ACCESS_KEY}}
SAUCE_USERNAME: ${{secrets.SAUCE_USERNAME}}
SAUCE_ACCESS_KEY: ${{ secrets.SAUCE_ACCESS_KEY }}
SAUCE_USERNAME: ${{ secrets.SAUCE_USERNAME }}

concurrency:
group: ${{ github.workflow }}-${{ github.head_ref || github.ref }}
Expand All @@ -24,8 +24,8 @@ jobs:
- name: Setup Node
uses: actions/setup-node@v4
with:
node-version-file: '.nvmrc'
cache: 'npm'
node-version-file: ".nvmrc"
cache: "npm"

- name: Install Dependencies
run: npm ci
Expand All @@ -45,8 +45,8 @@ jobs:
- name: Setup Node
uses: actions/setup-node@v4
with:
node-version-file: '.nvmrc'
cache: 'npm'
node-version-file: ".nvmrc"
cache: "npm"

- name: Install Dependencies
run: |
Expand All @@ -69,8 +69,8 @@ jobs:
- name: Setup Node
uses: actions/setup-node@v4
with:
node-version-file: '.nvmrc'
cache: 'npm'
node-version-file: ".nvmrc"
cache: "npm"

- name: Install Dependencies
run: |
Expand Down Expand Up @@ -98,13 +98,13 @@ jobs:
- name: Setup Node
uses: actions/setup-node@v4
with:
node-version-file: '.nvmrc'
cache: npm
node-version-file: ".nvmrc"
cache: "npm"

- name: Setup Python
uses: actions/setup-python@v5
with:
python-version: '3.11'
python-version: '3.12'

- name: Google Cloud Login
uses: 'google-github-actions/auth@v2'
Expand Down Expand Up @@ -146,17 +146,17 @@ jobs:
- name: Setup Node
uses: actions/setup-node@v4
with:
node-version-file: '.nvmrc'
cache: npm
node-version-file: ".nvmrc"
cache: "npm"

- name: Set up Python
uses: actions/setup-python@v5
with:
python-version: '3.11'
python-version: "3.12"

- name: Google Cloud Login
id: 'auth'
uses: 'google-github-actions/auth@v2'
id: "auth"
uses: google-github-actions/auth@v2
with:
credentials_json: '${{ secrets.GCS_RUNNER_SA_KEY }}'

Expand Down Expand Up @@ -236,7 +236,7 @@ jobs:
- name: Setup Python
uses: actions/setup-python@v5
with:
python-version: '3.11'
python-version: "3.12"

- name: Setup saucectl
uses: saucelabs/saucectl-run-action@v4
Expand All @@ -256,7 +256,7 @@ jobs:
username: ${{ secrets.SAUCE_USERNAME }}
accessKey: ${{ secrets.SAUCE_ACCESS_KEY }}
tunnelName: github-playwright-sc-check-tunnel-${{ matrix.os }}-${{ matrix.browser }}
scVersion: 4.9.1
scVersion: 4.9.2

- name: Test on Sauce
working-directory: ./tests/fixtures/cloud
Expand Down Expand Up @@ -286,8 +286,8 @@ jobs:
- name: Setup Node
uses: actions/setup-node@v4
with:
node-version-file: '.nvmrc'
cache: npm
node-version-file: ".nvmrc"
cache: "npm"

- name: Setup saucectl
uses: saucelabs/saucectl-run-action@v4
Expand Down
76 changes: 37 additions & 39 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
@@ -1,58 +1,56 @@
# Contributing to `sauce-playwright-runner`

**Thank you for your interest in `sauce-playwright-runner`. Your contributions are highly welcome.**
**Thank you for your interest in `sauce-playwright-runner`. Your contributions are highly valued.**

There are multiple ways of getting involved:
## Ways to Get Involved

- [Report a bug](#report-a-bug)
- [Suggest a feature](#suggest-a-feature)
- [Contribute code](#contribute-code)
- **[Report a Bug](#report-a-bug)**: Help improve the project by reporting bugs.
- **[Suggest a Feature](#suggest-a-feature)**: Propose new ideas and enhancements.
- **[Contribute Code](#contribute-code)**: Develop and contribute your code.

Below are a few guidelines we would like you to follow.
If you need help, please reach out to us by opening an issue.
Please follow the guidelines below to contribute effectively. If you need assistance, feel free to open an issue.

## Report a bug
Reporting bugs is one of the best ways to contribute. Before creating a bug report, please check that an [issue](/issues) reporting the same problem does not already exist. If there is such an issue, you may add your information as a comment.
## Report a Bug

To report a new bug you should open an issue that summarizes the bug and set the label to "bug".
Before reporting a bug, please ensure there is no existing issue addressing the same problem. If an issue exists, contribute by adding your information in a comment.

If you want to provide a fix along with your bug report: That is great! In this case please send us a pull request as described in section [Contribute Code](#contribute-code).
To report a new bug:
- Open an issue summarizing the bug.
- Label it as "bug".
- If you are also submitting a fix, please follow the instructions under [Contribute Code](#contribute-code).

## Suggest a feature
To request a new feature you should open an [issue](../../issues/new) and summarize the desired functionality and its use case. Set the issue label to "feature".
## Suggest a Feature

## Contribute code
This is an outline of what the workflow for code contributions looks like
For feature requests:
- Open a [new issue](../../issues/new).
- Clearly summarize the desired functionality and its use case.
- Label the issue as "feature".

- Check the list of open [issues](../../issues). Either assign an existing issue to yourself, or
create a new one that you would like work on and discuss your ideas and use cases.
## Contribute Code

It is always best to discuss your plans beforehand, to ensure that your contribution is in line with our goals.
### Workflow

- Fork the repository on GitHub
- Create a topic branch from where you want to base your work. This is usually 'main'.
- Open a new pull request, label it `work in progress` and outline what you will be contributing
- Make commits of logical units.
- Make sure you sign-off on your commits `git commit -s -m "adding X to change Y"`
- Write good commit messages (see below).
- Push your changes to a topic branch in your fork of the repository.
- As you push your changes, update the pull request with new infomation and tasks as you complete them
- Project maintainers might comment on your work as you progress
- When you are done, remove the `work in progess` label and ping the maintainers for a review
- Your pull request must receive a :thumbsup: from two [maintainers](MAINTAINERS)
1. Explore open [issues](../../issues). You can take up an existing issue or create a new one to discuss your ideas.
2. Fork the repository on GitHub.
3. Create a topic branch from `main`.
4. Start a pull request and label it as `work in progress`. Describe your contributions.
5. Commit your changes:
- Use logical commit units.
- Include clear commit messages.
- Sign off your commits with `git commit -s -m "describe your change"`.
6. Push your changes to your fork and update your pull request as you make progress.
7. Engage with any feedback from maintainers.
8. When complete, remove the `work in progress` label and notify maintainers for review.
9. Your pull request needs approval from two [maintainers](MAINTAINERS).

Thanks for your contributions!
### Commit Messages

### Commit messages
Your commit messages ideally can answer two questions: what changed and why. The subject line should feature the “what” and the body of the commit should describe the “why”.
Your commit messages should answer two questions: what changed and why? The subject should state what was changed, and the body should explain why.

When creating a pull request, its description should reference the corresponding issue id.
Reference the corresponding issue ID in your pull request description.

### Sign your work / Developer certificate of origin
All contributions (including pull requests) must agree to the Developer Certificate of Origin (DCO) version 1.1. This is exactly the same one created and used by the Linux kernel developers and posted on http://developercertificate.org/. This is a developer's certification that he or she has the right to submit the patch for inclusion into the project. Simply submitting a contribution implies this agreement, however, please include a "Signed-off-by" tag in every patch (this tag is a conventional way to confirm that you agree to the DCO) - you can automate this with a [Git hook](https://stackoverflow.com/questions/15015894/git-add-signed-off-by-line-using-format-signoff-not-working)

```
git commit -s -m "adding X to change Y"
```bash
git commit -s -m "your detailed description of the change"
```

**Have fun, and happy hacking!**
**Enjoy contributing, and happy coding!**
7 changes: 3 additions & 4 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
Sauce Playwright Runner
=======================
# Sauce Playwright Runner

Sauce Labs test runner image for [`saucectl`](https://github.com/saucelabs/saucectl) to run [Playwright](https://playwright.dev/) tests using [saucectl](https://docs.saucelabs.com/dev/cli/saucectl/). This repository contains the code that is being executed in Sauce Labs when running `saucectl run`.
This is the Sauce Labs test runner for [`saucectl`](https://github.com/saucelabs/saucectl), designed to run [Playwright](https://playwright.dev/) tests. The code in this repository is executed within Sauce Labs during a `saucectl run`.

If you are interested to contribute to this project, please have a look into our [contribution guidelines](https://github.com/saucelabs/sauce-playwright-runner/blob/main/CONTRIBUTING.md).
If you are interested in contributing to this project, please review our [contribution guidelines](https://github.com/saucelabs/sauce-playwright-runner/blob/main/CONTRIBUTING.md).
Loading

0 comments on commit 1416420

Please sign in to comment.