Skip to content

Brs cra5#143

Merged
paul-tan (paultan2021) merged 8 commits into
fork_cra5from
BRS_CRA5
Apr 26, 2022
Merged

Brs cra5#143
paul-tan (paultan2021) merged 8 commits into
fork_cra5from
BRS_CRA5

Conversation

@DannySu2021

Copy link
Copy Markdown

This PR will merge the not BRS features:

  1. Set packages private
  2. Add documents(Readme, ChangeLog, Migration)
  3. Update tasks(Shell scripts)
  4. Update GHA workflows

Comment thread .github/workflows/build-brs.yml Outdated
@@ -0,0 +1,50 @@
name: Build BRS

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Remove file build-brs.yml, Tao Fang is working on another PR to figure out how to build and test brs on CI

Comment thread tasks/e2e-installs.sh
# ******************************************************************************

cd "$temp_app_path"
npx create-react-app test-app-dist-tag --scripts-version=@latest

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Revert all changes in this file and keep it consistent with cra5
Tao Fang is working on how to build and test with CRA5

Comment thread tasks/e2e-kitchensink.sh Outdated
# Install the app in a temporary location
cd $temp_app_path
npx create-react-app test-kitchensink --template=file:"$root_path"/packages/react-scripts/fixtures/kitchensink
npx create-react-app test-kitchensink --scripts-version=@skyscanner/backpack-react-scripts --template=file:"$root_path"/packages/react-scripts/fixtures/kitchensink

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Revert all changes in this file, and keep it consistent with CRA5,
Tao Fang is working on how to build and test CRA5

Comment thread tasks/e2e-kitchensink-eject.sh Outdated
# Check for expected output
exists build/*.html
exists build/static/js/main.*.js
exists build/web/*.html

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Revert all changes in this file, and keep it consistent with CRA5,
Tao Fang is working on how to build and test CRA5

Comment thread tasks/e2e-simple.sh Outdated
exit 1
fi

if hash npm 2>/dev/null

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Revert all changes in this file, and keep it consistent with CRA5,
Tao Fang is working on how to build and test CRA5

Revert all changes in this file, and keep it consistent with CRA5,
Will create new workflow to build and test BRS

@paultan2021 paul-tan (paultan2021) left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@paultan2021 paul-tan (paultan2021) merged commit eac60f5 into fork_cra5 Apr 26, 2022
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.

2 participants