Skip to content

Restructure project and add Storybook #84

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

Closed
wants to merge 11 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
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
2 changes: 1 addition & 1 deletion .github/workflows/static-analysis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ jobs:
- name: Install yarn dependencies
run: yarn install
- name: Run ESLint
run: yarn run eslint nextjs thunderstore-components -f node_modules/@microsoft/eslint-formatter-sarif/sarif.js -o results.sarif || ":"
run: yarn run eslint packages -f node_modules/@microsoft/eslint-formatter-sarif/sarif.js -o results.sarif || ":"
- name: Upload SARIF
uses: github/codeql-action/upload-sarif@v1
with:
Expand Down
2 changes: 1 addition & 1 deletion .gitignore
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
.idea
/node_modules/
node_modules/
Copy link
Member

Choose a reason for hiding this comment

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

Each project currently ignores their own node_modules (other than the new storybook one). These should either be removed from the projects that include it, so it's clear that it is ignored at the root, or added to storybook and this change should be reverted.

**/yarn-error.log
39 changes: 39 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,39 @@
# Thunderstore UI

## Installation

Run `yarn install` at the root of the project, that's it.

## Repository structure

This repository is intended to function as a monorepo, which contains several
sub-projects inside it. The goal of this structure is to enable easier code
sharing between projects as well as more efficient and higher coverage test
suites

Tools used for enabling this behavior are primarily:

- yarn workspaces, as defined in the root `package.json`
- typescript configuration inheritance and path aliases
- tweaks for module resolution done in other tools such as nextjs or webpack,
making sure the path aliased packages are appropriately resolved
Comment on lines +16 to +19
Copy link
Member

Choose a reason for hiding this comment

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

These should probably start with capital letters


All sub-projects exist in the `packages` directory, includinig the following:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
All sub-projects exist in the `packages` directory, includinig the following:
All sub-projects exist in the `packages` directory, including the following:


### components

This project is intended to house common UI components that can be re-used
across different projects, e.g. the Thunderstore Mod Manager as well as the
website

### nextjs

This nextjs-based website project
Copy link
Member

Choose a reason for hiding this comment

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

This sounds incomplete?


### storybook

[storybook](https://storybook.js.org/) is used to make the component catalogue
easier to browse.

You can run it with `yarn storybook` within the storybook project, and access it
via localhost at the port provided on the terminal.
15 changes: 0 additions & 15 deletions nextjs/package.json

This file was deleted.

32 changes: 0 additions & 32 deletions nextjs/tsconfig.json

This file was deleted.

Loading