-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
refactor(infra): directory structure #4615
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Review or Edit in CodeSandboxOpen the branch in Web Editor • VS Code • Insiders |
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
☁️ Nx Cloud ReportCI is running/has finished running commands for commit 461849e. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this CI Pipeline Execution ✅ Successfully ran 2 targetsSent with 💌 from NxCloud. |
Deploying with
|
Latest commit: |
461849e
|
Status: | ✅ Deploy successful! |
Preview URL: | https://89465a46.affine-dev.pages.dev |
Branch Preview URL: | https://refactor-directory-structure.affine-dev.pages.dev |
4b63da9
to
742a4ab
Compare
24644aa
to
742a4ab
Compare
742a4ab
to
bf422ef
Compare
6def7e7
to
461849e
Compare
pengx17
approved these changes
Oct 18, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
docs
Improvements or additions to documentation
mod:component
mod:dev
mod:hooks
mod:native
mod:storage
mod:store
mod:workspace
Related to workspace
rust
test
Related to test cases
type:ui-style
Related to UI styling
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Redesign directory structure:
packages/
contains all code running in production.backend/
contains backend code, more information from https://github.com/toeverything/OctoBase.frontend/
contains frontend code, including the web app, the electron app and business libraries.plugins/
contains all build-in plugins.common
contains the isomorphic code or basic libraries without business.tools/
contains tools to help developing or CI, not used in production.tests/
contains testings across different libraries, including e2e testings and integration testings.Libraries movement:
storage
tobackend
this lib just used in affine backend businessgraphql
tofrontend
generate api code for affine fronthooks
tofrontend
separate packages to have more stricter specificationsnative
tofrontend
rust lib for electron appi18n
tofrontend
all local translations for affinetemplates
tofrontend
workspace initialization templatesworkspace
tofrontend
workspace storage logic for affinecli
totools
develop tool for affine monorepoplugin-cli
totools
develop tool for affine plugin, not public one nowworkers
totools
used in github workflowstorybook
totests
used for product acceptance in CI stage, as part of integration testingLibraries keeping in common temporarily:
debug
maybe removeenv
maybe removeinfra
not thought it through yet