Skip to content

Commit 9867885

Browse files
pyramationpyramation
authored
Merge pull request #181 from cosmology-tech/fix/ci-fail-to-detect
Fix CI test jobs fail to detect build failure
2 parents b8d4f72 + 68962c5 commit 9867885

File tree

261 files changed

+201364
-113734
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

261 files changed

+201364
-113734
lines changed

‎.github/workflows/run-tests-build.yaml‎

Lines changed: 82 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,82 @@
1+
name: Build Next.js examples
2+
3+
on:
4+
workflow_dispatch:
5+
pull_request:
6+
types: [opened, reopened, synchronize]
7+
8+
jobs:
9+
build:
10+
runs-on: ubuntu-latest
11+
strategy:
12+
max-parallel: 4
13+
fail-fast: true
14+
matrix:
15+
example:
16+
[
17+
asset-list,
18+
authz,
19+
vote-proposal,
20+
ibc-transfer,
21+
swap-tokens,
22+
provide-liquidity,
23+
nft,
24+
connect-chain,
25+
connect-multi-chain,
26+
]
27+
28+
steps:
29+
- name: Checkout repository
30+
uses: actions/checkout@v4
31+
32+
- name: Set template_path Environment Variable
33+
run: |
34+
example_value="${{ matrix.example }}"
35+
36+
if [[ "$example_value" == "connect-chain" ]] || [[ "$example_value" == "connect-multi-chain" ]]; then
37+
echo "template_path=templates" >> $GITHUB_ENV
38+
echo "ci_template_path=dest" >> $GITHUB_ENV
39+
else
40+
echo "template_path=examples" >> $GITHUB_ENV
41+
echo "ci_template_path=dest" >> $GITHUB_ENV
42+
fi
43+
44+
- name: Log
45+
run: |
46+
echo "Using template_path: $template_path"
47+
echo "Using ci_template_path: $ci_template_path"
48+
49+
- name: Clone example
50+
run: |
51+
echo "Cloning example: ./$template_path/${{ matrix.example }}/" into "${{ github.workspace }}/$ci_template_path/${{ matrix.example }}"
52+
53+
mkdir -p ${{ github.workspace }}/$ci_template_path/${{ matrix.example }}
54+
cp -r ./$template_path/${{ matrix.example }}/ ${{ github.workspace }}/$ci_template_path/
55+
cd ${{ github.workspace }}/$ci_template_path/${{ matrix.example }}
56+
57+
- name: Set up Node.js
58+
uses: actions/setup-node@v4
59+
with:
60+
node-version: 20
61+
62+
- name: Set up Yarn
63+
uses: threeal/[email protected]
64+
65+
- name: Cache NextJS
66+
uses: actions/cache@v4
67+
with:
68+
path: |
69+
${{ github.workspace }}/.next/cache
70+
# Generate a new cache whenever packages or source files change.
71+
key: ${{ runner.os }}-nextjs-${{ hashFiles('**/yarn.lock') }}-${{ hashFiles('**/*.js', '**/*.jsx', '**/*.ts', '**/*.tsx') }}
72+
# If source files changed but packages didn't, rebuild from a prior cache.
73+
restore-keys: |
74+
${{ runner.os }}-nextjs-${{ hashFiles('**/yarn.lock') }}-
75+
76+
- name: Build example project
77+
run: |
78+
echo "Building example: ${{ matrix.example }} in directory ${{ github.workspace }}/$ci_template_path/${{ matrix.example }}"
79+
cd ${{ github.workspace }}/$ci_template_path/${{ matrix.example }}
80+
ls -la
81+
yarn install
82+
yarn build

‎.github/workflows/run-tests.yaml‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
1-
name: Run Tests
1+
name: Run Tests Prod
22

33
on:
44
push:
5-
pull_request:
6-
types: [opened, synchronized, reopened]
5+
branches:
6+
- main
77
workflow_dispatch:
88

99
jobs:
@@ -18,23 +18,23 @@ jobs:
1818
- name: Setup Node.js
1919
uses: actions/setup-node@v4
2020
with:
21-
node-version: "20.x"
21+
node-version: '20.x'
2222

2323
- name: Install Dependencies
2424
run: npm install -g create-cosmos-app
2525

26+
- name: asset-list
27+
run: |
28+
cca --example asset-list --name asset-list
29+
cd asset-list
30+
yarn build
31+
2632
- name: vote-proposal
2733
run: |
2834
cca --example vote-proposal --name vote-proposal
2935
cd vote-proposal
3036
yarn build
3137
32-
# - name: asset-list
33-
# run: |
34-
# cca --example asset-list --name asset-list
35-
# cd asset-list
36-
# yarn build
37-
3838
# - name: ibc-transfer
3939
# run: |
4040
# cca --example ibc-transfer --name ibc-transfer

‎boilerplates/ts-codegen-npm-module/package.json‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,10 @@
5252
"publish-scripts": "0.1.0",
5353
"rimraf": "^5.0.0",
5454
"ts-jest": "^29.1.0",
55-
"typescript": "^5.0.4"
55+
"typescript": "^5.1.6"
5656
},
5757
"dependencies": {
5858
"@cosmjs/cosmwasm-stargate": "^0.31.0",
5959
"@cosmwasm/ts-codegen": "^0.31.6"
6060
}
61-
}
61+
}

‎boilerplates/website/package.json‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
"dependencies": {
2121
"@emotion/react": "11.10.6",
2222
"@emotion/styled": "11.10.6",
23-
"@interchain-ui/react": "^1.21.16",
23+
"@interchain-ui/react": "^1.23.16",
2424
"framer-motion": "9.0.7",
2525
"next": "^13",
2626
"react": "18.2.0",
@@ -34,6 +34,6 @@
3434
"eslint": "8.28.0",
3535
"eslint-config-next": "13.0.5",
3636
"generate-lockfile": "0.0.12",
37-
"typescript": "4.9.3"
37+
"typescript": "^5.1.6"
3838
}
3939
}

‎examples/asset-list/components/common/Layout.tsx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ export const Layout = ({ children }: { children?: React.ReactNode }) => {
1313
bottom="0"
1414
left="0"
1515
right="0"
16-
height="max(1dvh, 100%)"
16+
flex={1}
1717
backgroundColor={theme === 'light' ? '$white' : '$gray900'}
1818
data-part-id="layout-container"
1919
>

‎examples/asset-list/components/wallet/WalletConnect.tsx‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -56,16 +56,16 @@ export const ConnectWalletButton = ({
5656
gap="$4"
5757
as="span"
5858
borderRadius="8px"
59-
// @ts-ignore
60-
style={{
61-
zIndex: '1',
62-
}}
59+
zIndex={1}
60+
color={useColorModeValue('$textInverse', '$text')}
6361
>
6462
<Box as="span">
6563
<IoWallet />
6664
</Box>
6765

68-
<Box as="span">{buttonText ? buttonText : 'Connect Wallet'}</Box>
66+
<Box as="span" fontSize="$md" color="inherit">
67+
{buttonText ? buttonText : 'Connect Wallet'}
68+
</Box>
6969
</Box>
7070
</Button>
7171
);

‎examples/asset-list/package.json‎

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,8 @@
2727
"@cosmos-kit/react": "2.10.9",
2828
"@emotion/react": "11.10.6",
2929
"@emotion/styled": "11.10.6",
30-
"@interchain-ui/react": "^1.21.16",
30+
"@interchain-ui/react": "^1.23.16",
31+
"@interchain-ui/react-no-ssr": "^0.1.2",
3132
"@tanstack/react-query": "4.32.0",
3233
"@uidotdev/usehooks": "2.4.1",
3334
"bignumber.js": "^9.1.2",
@@ -40,18 +41,16 @@
4041
"osmo-query": "16.5.1",
4142
"react": "^18.2",
4243
"react-dom": "^18.2",
43-
"react-icons": "^4.12.0",
44-
"react-no-ssr": "1.1.0"
44+
"react-icons": "^4.12.0"
4545
},
4646
"devDependencies": {
4747
"@tanstack/react-query-devtools": "4.32.0",
4848
"@types/node": "^20.8.4",
4949
"@types/react": "^18.2.50",
5050
"@types/react-dom": "^18.2.18",
51-
"@types/react-no-ssr": "^1.1.7",
5251
"eslint": "^8.56.0",
5352
"eslint-config-next": "^14.1.0",
5453
"generate-lockfile": "0.0.12",
55-
"typescript": "^5"
54+
"typescript": "^5.1.6"
5655
}
5756
}

‎examples/asset-list/pages/_app.tsx‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,8 +69,6 @@ function CreateCosmosApp({ Component, pageProps }: AppProps) {
6969
>
7070
<QueryClientProvider client={queryClient}>
7171
<main id="main" className={themeClass}>
72-
{/* TODO fix type error */}
73-
{/* @ts-ignore */}
7472
<Component {...pageProps} />
7573
</main>
7674
</QueryClientProvider>

‎examples/asset-list/pages/multi-chain.tsx‎

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { useState } from 'react';
2-
import NoSSR from 'react-no-ssr';
2+
import { ReactNoSSR } from '@interchain-ui/react-no-ssr';
33
import { ChainName } from 'cosmos-kit';
44
import { AssetListSection, Layout, WalletSection } from '@/components';
55

@@ -13,10 +13,9 @@ export default function MultiChain() {
1313
providedChainName={chainName}
1414
setChainName={setChainName}
1515
/>
16-
{/* TODO fix type error */}
17-
{/* Type error: This JSX tag's 'children' prop expects a single child of type 'ReactNode', but multiple children were provided. */}
18-
{/* @ts-ignore */}
19-
<NoSSR>{chainName && <AssetListSection chainName={chainName} />}</NoSSR>
16+
<ReactNoSSR>
17+
{chainName && <AssetListSection chainName={chainName} />}
18+
</ReactNoSSR>
2019
</Layout>
2120
);
2221
}

‎examples/asset-list/pages/single-chain.tsx‎

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,14 @@
1-
import NoSSR from 'react-no-ssr';
21
import { defaultChainName } from '@/config';
2+
import { ReactNoSSR } from '@interchain-ui/react-no-ssr';
33
import { AssetListSection, Layout, WalletSection } from '@/components';
44

55
export default function SingleChain() {
66
return (
77
<Layout>
88
<WalletSection isMultiChain={false} />
9-
{/* TODO fix type error */}
10-
{/* Type error: This JSX tag's 'children' prop expects a single child of type 'ReactNode', but multiple children were provided. */}
11-
{/* @ts-ignore */}
12-
<NoSSR>
9+
<ReactNoSSR>
1310
<AssetListSection chainName={defaultChainName} />
14-
</NoSSR>
11+
</ReactNoSSR>
1512
</Layout>
1613
);
1714
}

0 commit comments

Comments
 (0)