difftreelog
Fixing spoted items.
in: master
5 files changed
.docker/Dockerfile-parachaindiffbeforeafterboth23RUN rustup target add wasm32-unknown-unknown --toolchain $RUST_TOOLCHAIN23RUN rustup target add wasm32-unknown-unknown --toolchain $RUST_TOOLCHAIN2424252526RUN mkdir unique_parachain26RUN mkdir /unique_parachain27WORKDIR /unique_parachain27WORKDIR /unique_parachain282829# ===== BUILD ======29# ===== BUILD ======32ARG PROFILE=release32ARG PROFILE=release33ARG FEATURE=33ARG FEATURE=343435RUN mkdir unique_parachain35RUN mkdir /unique_parachain36WORKDIR /unique_parachain36WORKDIR /unique_parachain373738RUN git clone $REPO_URL -b $BRANCH38RUN git clone $REPO_URL -b $BRANCH39RUN cargo build $FEATURE --$PROFILE39RUN cargo build $FEATURE --$PROFILE40 # && \41 # cargo test424043# ===== BUILD POLKADOT =====41# ===== BUILD POLKADOT =====44FROM rust-builder as builder-polkadot42FROM rust-builder as builder-polkadot454346ARG POLKA_VERSION=release-v0.9.2444ARG POLKA_VERSION=release-v0.9.2447ENV POLKA_VERSION $POLKA_VERSION45ENV POLKA_VERSION $POLKA_VERSION484649RUN mkdir unique_parachain47RUN mkdir /unique_parachain50WORKDIR /unique_parachain48WORKDIR /unique_parachain514952RUN git clone -b $POLKA_VERSION --depth 1 https://github.com/paritytech/polkadot.git && \50RUN git clone -b $POLKA_VERSION --depth 1 https://github.com/paritytech/polkadot.git && \77COPY --from=builder-unique /unique_parachain/target/release/unique-collator /unique-chain/target/release/75COPY --from=builder-unique /unique_parachain/target/release/unique-collator /unique-chain/target/release/78COPY --from=builder-polkadot /unique_parachain/polkadot/target/release/polkadot /polkadot/target/release/76COPY --from=builder-polkadot /unique_parachain/polkadot/target/release/polkadot /polkadot/target/release/797780COPY ["./launch-config.json", "/polkadot-launch/launch-config.json"]78COPY --from=builder-unique /unique_parachain/launch-config.json /polkadot-launch/launch-config.json8179828083CMD export NVM_DIR="$HOME/.nvm" && \81CMD export NVM_DIR="$HOME/.nvm" && \.github/workflows/codestyle.ymldiffbeforeafterboth--- a/.github/workflows/codestyle.yml
+++ b/.github/workflows/codestyle.yml
@@ -14,6 +14,24 @@
runs-on: self-hosted-ci
steps:
+ - name: Skip if pull request is in Draft
+ # `if: github.event.pull_request.draft == true` should be kept here, at
+ # the step level, rather than at the job level. The latter is not
+ # recommended because when the PR is moved from "Draft" to "Ready to
+ # review" the workflow will immediately be passing (since it was skipped),
+ # even though it hasn't actually ran, since it takes a few seconds for
+ # the workflow to start. This is also disclosed in:
+ # https://github.community/t/dont-run-actions-on-draft-pull-requests/16817/17
+ # That scenario would open an opportunity for the check to be bypassed:
+ # 1. Get your PR approved
+ # 2. Move it to Draft
+ # 3. Push whatever commits you want
+ # 4. Move it to "Ready for review"; now the workflow is passing (it was
+ # skipped) and "Check reviews" is also passing (it won't be updated
+ # until the workflow is finished)
+ if: github.event.pull_request.draft == true
+ run: exit 1
+
- uses: actions/checkout@v1
- name: Install latest nightly
uses: actions-rs/toolchain@v1
@@ -30,6 +48,24 @@
runs-on: self-hosted-ci
steps:
+ - name: Skip if pull request is in Draft
+ # `if: github.event.pull_request.draft == true` should be kept here, at
+ # the step level, rather than at the job level. The latter is not
+ # recommended because when the PR is moved from "Draft" to "Ready to
+ # review" the workflow will immediately be passing (since it was skipped),
+ # even though it hasn't actually ran, since it takes a few seconds for
+ # the workflow to start. This is also disclosed in:
+ # https://github.community/t/dont-run-actions-on-draft-pull-requests/16817/17
+ # That scenario would open an opportunity for the check to be bypassed:
+ # 1. Get your PR approved
+ # 2. Move it to Draft
+ # 3. Push whatever commits you want
+ # 4. Move it to "Ready for review"; now the workflow is passing (it was
+ # skipped) and "Check reviews" is also passing (it won't be updated
+ # until the workflow is finished)
+ if: github.event.pull_request.draft == true
+ run: exit 1
+
- uses: actions/checkout@v3
- name: Install substrate dependencies
run: sudo apt-get install libssl-dev pkg-config libclang-dev clang
.github/workflows/node_build_test.ymldiffbeforeafterboth--- a/.github/workflows/node_build_test.yml
+++ b/.github/workflows/node_build_test.yml
@@ -74,7 +74,7 @@
RUST_TOOLCHAIN=${{ env.RUST_TOOLCHAIN }}
POLKA_VERSION=${{ env.POLKA_VERSION }}
FEATURE=${{ matrix.features }}
- BRANCH=${{ github.head_ref }}
+ BRANCH=${{ github.event.pull_request.head.sha }}
- name: Show build configuration
run: cat .docker/docker-compose.${{ matrix.network }}.yml
@@ -91,7 +91,6 @@
cd tests
yarn install
yarn add mochawesome
- yarn --pure-lockfile
echo "Ready to start tests"
node scripts/readyness.js
NOW=$(date +%s) && yarn test --reporter mochawesome --reporter-options reportFilename=test-${NOW}
.github/workflows/tests_codestyle.ymldiffbeforeafterboth--- a/.github/workflows/tests_codestyle.yml
+++ b/.github/workflows/tests_codestyle.yml
@@ -13,13 +13,31 @@
runs-on: self-hosted-ci
steps:
- - uses: actions/checkout@v3
-
- - uses: actions/setup-node@v3
- with:
- node-version: 16
+ - name: Skip if pull request is in Draft
+ # `if: github.event.pull_request.draft == true` should be kept here, at
+ # the step level, rather than at the job level. The latter is not
+ # recommended because when the PR is moved from "Draft" to "Ready to
+ # review" the workflow will immediately be passing (since it was skipped),
+ # even though it hasn't actually ran, since it takes a few seconds for
+ # the workflow to start. This is also disclosed in:
+ # https://github.community/t/dont-run-actions-on-draft-pull-requests/16817/17
+ # That scenario would open an opportunity for the check to be bypassed:
+ # 1. Get your PR approved
+ # 2. Move it to Draft
+ # 3. Push whatever commits you want
+ # 4. Move it to "Ready for review"; now the workflow is passing (it was
+ # skipped) and "Check reviews" is also passing (it won't be updated
+ # until the workflow is finished)
+ if: github.event.pull_request.draft == true
+ run: exit 1
+
+ - uses: actions/checkout@v3
+
+ - uses: actions/setup-node@v3
+ with:
+ node-version: 16
- - name: Install modules
- run: cd tests && yarn
- - name: Run ESLint
- run: cd tests && yarn eslint --ext .ts,.js src/
+ - name: Install modules
+ run: cd tests && yarn
+ - name: Run ESLint
+ run: cd tests && yarn eslint --ext .ts,.js src/
tests/scripts/readyness.jsdiffbeforeafterboth--- a/tests/scripts/readyness.js
+++ b/tests/scripts/readyness.js
@@ -24,7 +24,7 @@
break;
}
catch(e) {
- await sleep(60000);
+ await sleep(10000);
}
}
}