Skip to content
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

Improve tooling of reactist #675

Open
1 of 9 tasks
gnapse opened this issue Jul 26, 2022 · 0 comments
Open
1 of 9 tasks

Improve tooling of reactist #675

gnapse opened this issue Jul 26, 2022 · 0 comments

Comments

@gnapse
Copy link
Contributor

gnapse commented Jul 26, 2022

⁉️ Why

  • To make it easier to make new releases with fixes and new features
  • To make it easier to work with Reactist locally, in development mode, alongside our product apps
  • To bring Reactist build process closer to how our apps are built, and support the same bundling features

🗺 Overview

  • Releasing new Reactist versions is cumbersome (requires manual update of changelog, manual update of new version number, generating built artifacts, etc.)
  • Working with Reactist locally against our product apps is difficult and error-prone. Although not stated below explicitly in any given task, hopefully an improved build process could help in this regard. For instance, right now stylesheet changes in Reactist do not reflect in our apps running locally against the local dev version of Reactist.
  • Reactist build process and bundling features are not 100% compatible with that of our apps. We want to support the same features (e.g. importing SVGs as React components, support the same types of stylesheet imports into JS code, etc.)

✅ Task Breakdown

  1. Build Process

    We want to unify and simplify our build process while making it faster if possible. This includes:

    • Remove built artifacts from source control (Do we need built assets in git #252)
    • Switch off from TSDX to another setup (webpack? esbuild? Something else?)
    • Support tree-shaking. Apps and other libraries using Reactist must only include strictly what they are using from it, and nothing more.
    • Support TypeScript, CSS modules and SVGs imported as React components. It must also support .less while we still rely on it, as we gradually move away from it.
  2. Automated project life-cycle

    Implement automation of versioning and release. This could be achieved drawing inspiration, for instance, from how jest-dom does it (powered internally by cycjimmy/semantic-release-action).

    This includes:

    • Having actions that make sure that all commits and/or PR titles follow the Angular Commit Message Conventions.
    • The release automation draws conclusions from the commits/PRs in the release, to determine if the release is a patch, minor or major (breaking change) release.
    • Push actions to main trigger a new release fully automatically, including determining the new version.
    • Automated release of storybooks to https://doist.github.io/reactist/
  3. PR review process aids

🚧 Potential problems / unknowns

  • We may not be able to fit all this work within a single cycle (4-5 weeks).

    The numbered list above is in order of priority. Hopefully all the checkboxes within each section are done, and no numbered section is left halfway, as they are more or less self-contained and all tasks within each are interconnected.

🔮 Out of scope / future improvements

📝 Related documents & discussion

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant