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

fix(pie-monorepo): DSW-1969 updated Playwright and Percy dependencies OLD #1600

Closed
wants to merge 37 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
37 commits
Select commit Hold shift + click to select a range
5fc2a27
refactor(pie-monorepo): DSW-1969 updated tests and playwright packages
JoshuaNg2332 Jul 4, 2024
d7edba0
chore(pie-monorepo): DSW-1969 changesets file
JoshuaNg2332 Jul 5, 2024
f743363
chore(pie-monorepo): DSW-1969 yarn update
JoshuaNg2332 Jul 12, 2024
e4d07e4
chore(deps): dsw-000 update dependency @rollup/plugin-commonjs to v25…
renovate[bot] Jul 12, 2024
80c5468
fix(pie-assistive-text): DSW-1969 updated visual test to fix promise …
JoshuaNg2332 Jul 15, 2024
7316310
format(pie-modal): DSW-1969 added comment for ticket to address
JoshuaNg2332 Jul 15, 2024
27a6e73
format(pie-button): DSW-1969 removed unneeded eslint line and updated…
JoshuaNg2332 Jul 15, 2024
619902f
Merge branch 'main' into DSW-1969-percy-cli-chrome-fix-5
JoshuaNg2332 Jul 15, 2024
0cc5d19
docs(pie-docs): DSW-1624 update pie-webc documentation for pie-docs (…
leksaBoiko Jul 16, 2024
bdda830
Merge branch 'main' into DSW-1969-percy-cli-chrome-fix-5
JoshuaNg2332 Jul 16, 2024
ada2869
chore(pie-monorepo): DSW-1969 updated percy/cli package
JoshuaNg2332 Jul 17, 2024
63709c3
Merge branch 'main' into DSW-1969-percy-cli-chrome-fix-5
JoshuaNg2332 Jul 17, 2024
f61b574
ci(pie-monorepo): DSW-1969 updated ci.yml for logging
JoshuaNg2332 Jul 18, 2024
2f960e1
ci(pie-monorepo): DSW-1969 percy debug env variables added
JoshuaNg2332 Jul 25, 2024
ba25292
feat(pie-checkbox-group): DSW-2182 main component functionality (#1581)
dandel10n Jul 18, 2024
6824d19
chore(deps): dsw-000 update dependency @babel/cli to v7.24.8 (#1602)
renovate[bot] Jul 18, 2024
3b6b293
chore(pie-monorepo): DSW-000 let commitizen read the ticket number fr…
xander-marjoram Jul 18, 2024
2cb6919
fix(pie-cookie-banner): WCP-1109 update cookie banner copy for en-FR …
xander-marjoram Jul 18, 2024
6cedb3c
feat(pie-monorepo): DSW-2056 added pie-webc integration to next 10 ex…
kevinrodrigues Jul 22, 2024
201307c
feat(pie-monorepo): DSW-2056 pie webc integration next13 (#1613)
kevinrodrigues Jul 22, 2024
776dc10
docs(pie-docs): DSW-2211 adding overview of web components (#1589)
maledr5 Jul 22, 2024
667b174
docs(pie-docs): DSW-2220 add overview page and assets (#1605)
leksaBoiko Jul 23, 2024
5df20f5
feat(pie-cookie-banner): DSW-2210 add entrance animation variants (#1…
xander-marjoram Jul 23, 2024
ebd078c
Version Packages (#1606)
pie-design-system-bot Jul 23, 2024
0483fff
feat(pie-monorepo): DSW-2056 integrate pie-webc into react18 examples…
kevinrodrigues Jul 23, 2024
bb4962b
chore(deps): dsw-000 update dependency @babel/core to v7.24.9 (#1620)
renovate[bot] Jul 24, 2024
04b380e
chore(deps): dsw-000 update actions/upload-artifact action to v4.3.4 …
renovate[bot] Jul 24, 2024
61c9cee
feat(pie-monorepo): DSW-2056 added pie-webc integration to react 17 e…
kevinrodrigues Jul 24, 2024
4223816
feat(pie-button): DSW-000 Changing styles to use classmap (#1587)
ashleynolan Jul 24, 2024
98dfca0
feat(pie-checkbox-group): DSW-2182 checkbox group styling (#1618)
dandel10n Jul 24, 2024
1fc27b0
feat(pie-webc): DSW-2182 isRtl attr changed to data-is-rtl (#1625)
dandel10n Jul 24, 2024
774a7ec
Version Packages (#1623)
pie-design-system-bot Jul 25, 2024
e7965d7
refactor(pie-monorepo): DSW-1969 updated tests and playwright packages
JoshuaNg2332 Jul 4, 2024
b1c800d
chore(pie-monorepo): DSW-1969 yarn update
JoshuaNg2332 Jul 12, 2024
70a3134
docs(pie-docs): DSW-1624 update pie-webc documentation for pie-docs (…
leksaBoiko Jul 16, 2024
e7d2e7a
chore(pie-monorepo): DSW-1969 updated percy/cli package
JoshuaNg2332 Jul 17, 2024
9571fb7
chore(pie-monorepo): DSW-1969 update yarn
JoshuaNg2332 Jul 25, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 0 additions & 5 deletions .changeset/silver-dryers-attack.md

This file was deleted.

5 changes: 5 additions & 0 deletions .changeset/slow-chicken-allow.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"pie-monorepo": patch
---

Updated Playwright and Percy CLI dependancies
11 changes: 9 additions & 2 deletions .cz-config.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
const fs = require('fs')
const { globSync } = require('glob')


const getPackages = () => {
const ignore = [
'apps/examples/**',
Expand All @@ -22,6 +21,13 @@ const getPackages = () => {
return packageNames;
};

const getCurrentTicketNumberFromBranch = () => {
const branchName = require('child_process').execSync('git rev-parse --abbrev-ref HEAD').toString().trim();
const ticketNumber = branchName.match(/\d{1,7}/)?.[0];

return ticketNumber;
};

module.exports = {
// We have to provide '\u0020' to some emojis due to a spacing bug with commitizen - https://github.com/commitizen/cz-cli/issues/815
types: [
Expand All @@ -45,7 +51,7 @@ module.exports = {
messages: {
type: "Select the type of change that you're committing:",
scope: 'Denote the SCOPE of this change:',
ticketNumber: "Jira ticket number (enter 0 to fill automatically):",
ticketNumber: `Jira ticket number (${getCurrentTicketNumberFromBranch() || '000'}):`,
subject: 'Write a SHORT, IMPERATIVE tense description of the change:',
body: '(optional) Provide a LONGER description of the change. Use "|" to break new line:',
breaking: '(optional) List any BREAKING CHANGES:',
Expand All @@ -55,6 +61,7 @@ module.exports = {
allowCustomScopes: false,
skipQuestions: ['footer'],
allowTicketNumber: true,
fallbackTicketNumber: getCurrentTicketNumberFromBranch() || '000',
isTicketNumberRequired: true,
ticketNumberPrefix: 'DSW-',
ticketNumberRegExp: '\\d{1,7}',
Expand Down
5 changes: 5 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,11 @@ env:
PERCY_TOKEN_PIE_TEXTAREA: ${{ secrets.PERCY_TOKEN_PIE_TEXTAREA }}
PERCY_TOKEN_PIE_TOAST: ${{ secrets.PERCY_TOKEN_PIE_TOAST }}
TURBO_TOKEN: ${{ secrets.TURBO_TOKEN }}
PERCY_DEBUG: "*"
LOG_LEVEL: debug
PERCY_DO_NOT_CAPTURE_RESPONSIVE_ASSETS: true
PERCY_PAGE_LOAD_TIMEOUT: 60000
PERCY_NETWORK_IDLE_WAIT_TIMEOUT: 60000

jobs:
dependency-changeset:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/ossf-scorecard.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ jobs:
results_format: sarif

- name: Upload artifact
uses: actions/upload-artifact@65462800fd760344b1a7b4382951275a0abb4808 # v4.3.3
uses: actions/upload-artifact@0b2256b8c012f0828dc542b3febcab082c67f72b # v4.3.4
with:
name: SARIF
path: results.sarif
Expand Down
4 changes: 4 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -176,3 +176,7 @@ custom-elements.json
**/src/react.ts
**/stats.html
**/component-statuses.json

# Yalc
.yalc
yalc.lock
6 changes: 6 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,11 @@
# Changelog

## 2.62.0

### Minor Changes

- [Changed] - Let commitizen read ticket number from branch name ([#1614](https://github.com/justeattakeaway/pie/pull/1614)) by [@xander-marjoram](https://github.com/xander-marjoram)

## 2.61.3

### Patch Changes
Expand Down
18 changes: 18 additions & 0 deletions apps/examples/wc-angular12/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,23 @@
# wc-angular12

## 0.5.1

### Patch Changes

- Updated dependencies [[`927b800ed`](https://github.com/justeattakeaway/pie/commit/927b800ed2456c6db299694fbf850dea4aed34bd)]:
- @justeattakeaway/[email protected]

## 0.5.0

### Minor Changes

- [Added] pie-webc integration to angular examples app ([#1611](https://github.com/justeattakeaway/pie/pull/1611)) by [@kevinrodrigues](https://github.com/kevinrodrigues)

### Patch Changes

- Updated dependencies []:
- @justeattakeaway/[email protected]

## 0.4.43

### Patch Changes
Expand Down
4 changes: 2 additions & 2 deletions apps/examples/wc-angular12/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "wc-angular12",
"version": "0.4.43",
"version": "0.5.1",
"private": true,
"scripts": {
"ng": "ng",
Expand All @@ -18,7 +18,7 @@
"@angular/platform-browser-dynamic": "15.2.0",
"@angular/router": "15.2.0",
"@justeattakeaway/pie-css": "0.12.1",
"@justeattakeaway/pie-webc": "0.5.13",
"@justeattakeaway/pie-webc": "0.5.15",
"rxjs": "7.8.0",
"tslib": "2.3.0",
"zone.js": "0.12.0"
Expand Down
3 changes: 1 addition & 2 deletions apps/examples/wc-next10/ButtonSizes.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
import React from 'react';
import { sizes } from '@justeattakeaway/pie-button';
import { PieButton } from '@justeattakeaway/pie-button/dist/react';
import { PieButton, sizes } from '@justeattakeaway/pie-webc/react/button';

export default function ButtonSizes () {
return (
Expand Down
18 changes: 18 additions & 0 deletions apps/examples/wc-next10/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,23 @@
# wc-next10

## 0.6.1

### Patch Changes

- Updated dependencies [[`927b800ed`](https://github.com/justeattakeaway/pie/commit/927b800ed2456c6db299694fbf850dea4aed34bd)]:
- @justeattakeaway/[email protected]

## 0.6.0

### Minor Changes

- [Added] pie-webc integration to next10 example app ([#1617](https://github.com/justeattakeaway/pie/pull/1617)) by [@kevinrodrigues](https://github.com/kevinrodrigues)

### Patch Changes

- Updated dependencies []:
- @justeattakeaway/[email protected]

## 0.5.46

### Patch Changes
Expand Down
2 changes: 1 addition & 1 deletion apps/examples/wc-next10/Counter.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import React, { useState } from 'react';
import { PieButton } from '@justeattakeaway/pie-button/dist/react';
import { PieButton } from '@justeattakeaway/pie-webc/react/button';

export default function Counter () {
const [count, setCount] = useState(0);
Expand Down
3 changes: 1 addition & 2 deletions apps/examples/wc-next10/VariantSwitch.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
import React, { useState, useRef } from 'react';
import { variants } from '@justeattakeaway/pie-button';
import { PieButton } from '@justeattakeaway/pie-button/dist/react';
import { PieButton, variants } from '@justeattakeaway/pie-webc/react/button';

export default function VariantSwitch () {
const variantIndex = useRef(0);
Expand Down
1 change: 1 addition & 0 deletions apps/examples/wc-next10/next.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ const path = require('path');

const transpileModules = [
'@justeattakeaway/pie-button',
'@justeattakeaway/pie-webc',
'@justeattakeaway/pie-cookie-banner',
'@lit/react',
'lit'
Expand Down
5 changes: 2 additions & 3 deletions apps/examples/wc-next10/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "wc-next10",
"version": "0.5.46",
"version": "0.6.1",
"private": true,
"scripts": {
"dev": "next dev",
Expand All @@ -10,9 +10,8 @@
"lint:examples": "eslint ."
},
"dependencies": {
"@justeattakeaway/pie-button": "0.47.8",
"@justeattakeaway/pie-cookie-banner": "0.21.0",
"@justeattakeaway/pie-css": "0.12.1",
"@justeattakeaway/pie-webc": "0.5.15",
"@lit/react": "1.0.2",
"next": "10.2.3",
"next-transpile-modules": "4.1.0",
Expand Down
21 changes: 21 additions & 0 deletions apps/examples/wc-next13/.eslintrc.cjs
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
module.exports = {
root: true,
env: { browser: true, es2020: true },
extends: [
'eslint:recommended',
'plugin:react/recommended',
'plugin:react/jsx-runtime',
'plugin:react-hooks/recommended',
],
ignorePatterns: ['dist', '.eslintrc.cjs'],
parserOptions: { ecmaVersion: 'latest', sourceType: 'module' },
settings: { react: { version: '18.2' } },
plugins: ['react-refresh'],
rules: {
'react/jsx-no-target-blank': 'off',
'react-refresh/only-export-components': [
'warn',
{ allowConstantExport: true },
],
},
}
61 changes: 21 additions & 40 deletions apps/examples/wc-next13/.gitignore
Original file line number Diff line number Diff line change
@@ -1,43 +1,24 @@
# See https://help.github.com/articles/ignoring-files/ for more about ignoring files.

# dependencies
/node_modules
/.pnp
.pnp.js

.yarn/*
!.yarn/patches
!.yarn/plugins
!.yarn/releases
!.yarn/sdks
!.yarn/versions

# testing
/coverage

# next.js
/.next/
/out/

# production
/build

# misc
.DS_Store
*.pem

# debug
# Logs
logs
*.log
npm-debug.log*
yarn-debug.log*
yarn-error.log*
.pnpm-debug.log*

# local env files
.env*.local

# vercel
.vercel

# typescript
*.tsbuildinfo
next-env.d.ts
pnpm-debug.log*
lerna-debug.log*

node_modules
dist
dist-ssr
*.local

# Editor directories and files
.vscode/*
!.vscode/extensions.json
.idea
.DS_Store
*.suo
*.ntvs*
*.njsproj
*.sln
*.sw?
18 changes: 18 additions & 0 deletions apps/examples/wc-next13/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,23 @@
# wc-next13

## 0.4.1

### Patch Changes

- Updated dependencies [[`927b800ed`](https://github.com/justeattakeaway/pie/commit/927b800ed2456c6db299694fbf850dea4aed34bd)]:
- @justeattakeaway/[email protected]

## 0.4.0

### Minor Changes

- [Added] - pie-webc integration to next13 example apps ([#1613](https://github.com/justeattakeaway/pie/pull/1613)) by [@kevinrodrigues](https://github.com/kevinrodrigues)

### Patch Changes

- Updated dependencies []:
- @justeattakeaway/[email protected]

## 0.3.43

### Patch Changes
Expand Down
11 changes: 9 additions & 2 deletions apps/examples/wc-next13/README.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,11 @@
## How to use Next.js App
# React + Vite

This template provides a minimal setup to get React working in Vite with HMR and some ESLint rules.

Currently, two official plugins are available:

- [@vitejs/plugin-react](https://github.com/vitejs/vite-plugin-react/blob/main/packages/plugin-react/README.md) uses [Babel](https://babeljs.io/) for Fast Refresh
- [@vitejs/plugin-react-swc](https://github.com/vitejs/vite-plugin-react-swc) uses [SWC](https://swc.rs/) for Fast Refresh

### `yarn dev`

Expand Down Expand Up @@ -39,4 +46,4 @@ React 18 and previous versions don't handle web components and custom elements o

Therefore, in order to import the web component into your React application, please use:

import { PieButton } from '@justeattakeaway/pie-button/dist/react';
import { PieButton } from '@justeattakeaway/pie-webc/react/button.js';
13 changes: 13 additions & 0 deletions apps/examples/wc-next13/index.html
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
<!doctype html>
<html lang="en">
<head>
<meta charset="UTF-8" />
<link rel="icon" type="image/svg+xml" href="/vite.svg" />
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
<title>Vite + React</title>
</head>
<body>
<div id="root"></div>
<script type="module" src="/src/main.jsx"></script>
</body>
</html>
5 changes: 5 additions & 0 deletions apps/examples/wc-next13/next-env.d.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
/// <reference types="next" />
/// <reference types="next/image-types/global" />

// NOTE: This file should not be edited
// see https://nextjs.org/docs/basic-features/typescript for more information.
Loading
Loading