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

refactor: Merge workspace into single package. #362

Merged
merged 11 commits into from
Feb 6, 2023
Merged

refactor: Merge workspace into single package. #362

merged 11 commits into from
Feb 6, 2023

Conversation

FabianLars
Copy link
Member

There are no logic changes other than added support for pnpm detection. While i was at it i changed the package manager to pnpm like we did with the plugins repo.

The intend of this PR is to make the repo easier to maintain, use, and contribute to.

This PR should serve as an interim step, or an alternative to #355 because imo this action has too many users to remove most of it's feature before v1, but at the same time i think it's not in the state to be v1 -> the latter i'm trying to solve so that we can have a stable v1 and then we can look at Jonas' PR for v2 or v3, or maybe opt into a completely separate action.

@FabianLars
Copy link
Member Author

@lucasfernog any last arguments to convince me not to do it? :D
seriously though, i go crazy maintaining the action, and i'd like to go through with this before more people open (or try and fail to open) new PRs and before i tackle some other outstanding issues.

@lucasfernog
Copy link
Member

Just do it :D let's see how it goes.

@FabianLars FabianLars merged commit c5c0e27 into dev Feb 6, 2023
@FabianLars FabianLars deleted the refactor branch February 6, 2023 12:45
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

Successfully merging this pull request may close these issues.

None yet

2 participants