Skip to content

Conversation

@NullVoxPopuli
Copy link
Collaborator

@NullVoxPopuli NullVoxPopuli commented May 30, 2022

I'm splitting certain things out of the #144, because it's very big, and since we all are part-time maintainers of this library, I'm trying to make the individual parts more easily reviewable.

Some advantages of monorepo (as it relates to addon development)

  • we'll know:
    • which deps and files are for the docs
    • which deps and files are for testing
    • (later) which deps and files are specifically for the addon itself

This allows for more confidence in dependency changes.

For ease of reviewing, this is just the first step -- moving the addon directory into a sub folder -- docs and tests will be extracted in a separate PR

@NullVoxPopuli NullVoxPopuli force-pushed the convert-to-monorepo branch 2 times, most recently from 01034e5 to aa25ad5 Compare June 11, 2022 00:59
@NullVoxPopuli NullVoxPopuli marked this pull request as ready for review June 11, 2022 01:24
@NullVoxPopuli NullVoxPopuli added the internal Internal changes that still require a mention in the changelog/release notes label Jun 12, 2022
@NullVoxPopuli NullVoxPopuli merged commit d43c2f7 into master Jun 12, 2022
@NullVoxPopuli NullVoxPopuli deleted the convert-to-monorepo branch June 12, 2022 10:48
This was referenced Jun 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Internal changes that still require a mention in the changelog/release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants