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

Bump deps #158

Merged
merged 11 commits into from
Jul 28, 2024
Merged

Bump deps #158

merged 11 commits into from
Jul 28, 2024

Conversation

lishaduck
Copy link
Contributor

So that I can run turbo watch build. Bumped the rest while I was at it.

Copy link

changeset-bot bot commented Jul 9, 2024

🦋 Changeset detected

Latest commit: 55e2a20

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 8 packages
Name Type
create-sheriff-config Patch
eslint-config-sheriff Patch
@sherifforg/constants Patch
sheriff-webservices Patch
@sherifforg/types Patch
@sheriff/utils Patch
docs-website Patch
tsconfig Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

vercel bot commented Jul 9, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sheriff ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 28, 2024 10:38pm

@lishaduck
Copy link
Contributor Author

lishaduck commented Jul 9, 2024

I bumped everything that wouldn't make external consumers need to bump.
I guess if #157 & #156 are majors, then it doesn't matter, but I left that up to you.

@lishaduck
Copy link
Contributor Author

Oh, and I also noticed some publint complaints, so I bumped tsup and fixed your bundle.

.prettierignore Outdated Show resolved Hide resolved
.vscode/settings.json Outdated Show resolved Hide resolved
.vscode/settings.json Outdated Show resolved Hide resolved
turbo.json Outdated Show resolved Hide resolved
packages/sheriff-types/package.json Outdated Show resolved Hide resolved
packages/eslint-config-sheriff/package.json Outdated Show resolved Hide resolved
package.json Show resolved Hide resolved
@AndreaPontrandolfo
Copy link
Owner

Also we are missing the changeset here

@AndreaPontrandolfo AndreaPontrandolfo self-requested a review July 21, 2024 11:44
@lishaduck
Copy link
Contributor Author

lishaduck commented Jul 23, 2024

Should typesync v0.13.0 be included? It's an easy migration (nothing breaking, hopefully. If so, that's a bug.). I added native PNPM monorepo support, so it'd require tweaking tasks though and this PI is already big. And I don't really want to clone Sheriff on this machine.

@AndreaPontrandolfo
Copy link
Owner

I saw your PR, it's great, we use typesync at my workplace. I'm gonna try the update!
For Sheriff, yeah, I'm gonna add it to the backlog, we'll see.

@lishaduck
Copy link
Contributor Author

I saw your PR, it's great, we use typesync at my workplace. I'm gonna try the update! For Sheriff, yeah, I'm gonna add it to the backlog, we'll see.

Ok, I don't really care when it gets bumped, but if you do bump, I'd suggest basing it on this branch.

@AndreaPontrandolfo AndreaPontrandolfo merged commit bde2124 into AndreaPontrandolfo:master Jul 28, 2024
3 checks passed
@lishaduck lishaduck deleted the bump-deps branch July 29, 2024 18:08
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.

2 participants