use-ink / contracts-ui

Web application for deploying wasm smart contracts on Substrate chains that include the FRAME contracts pallet
https://contracts-ui.substrate.io/
GNU General Public License v3.0
61 stars 44 forks source link

chore: new file naming convention #514

Closed gaboesquivel closed 8 months ago

gaboesquivel commented 10 months ago

Lowercase, dash-separated folder names, such as components/auth-wizard. Within these folders, adhere to consistent file naming conventions, including extensions like:

Eg: import { NftItem } from './nft-item

└── nft-item
    ├── index.ts   ( exports )
    ├── nft-item.tsx
    ├── nft-item-header.tsx
    ├── nft-item-footer.tsx
    ├── nft-item-main.tsx
    ├── use-nft-item.ts
    ├── nft-item.type.ts
    ├── nft-item.context.tsx
    └── nft-item.test.tsx

github checkout action fails due this issue. https://github.com/paritytech/contracts-ui/pull/514

https://dev.to/peaceiris/the-case-sensitivity-of-the-github-actions-runner-3e9f

I suppose once this is merged the next pr will work well. All tests are passing locally.

image

netlify[bot] commented 10 months ago

Deploy Preview for contracts-ui failed.

Name Link
Latest commit 5c910a78e2904c7371e8cff0e9791dc412354939
Latest deploy log https://app.netlify.com/sites/contracts-ui/deploys/64f6868deacb760008937422
cypress[bot] commented 10 months ago

Passing run #228 ↗︎

0 61 0 0 Flakiness 0
⚠️ You've recorded test results over your free plan limit.
Upgrade your plan to view test results.

Details:

chore: new file naming convention
Project: Contracts UI Commit: 5c910a78e2
Status: Passed Duration: 02:06 💡
Started: Sep 5, 2023 1:59 AM Ended: Sep 5, 2023 2:01 AM

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.