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

Fix tiny Tailwind config details #82

Merged
merged 3 commits into from
Apr 20, 2022
Merged

Fix tiny Tailwind config details #82

merged 3 commits into from
Apr 20, 2022

Conversation

mqp
Copy link
Collaborator

@mqp mqp commented Apr 19, 2022

No functionality change, but having the old Tailwind 2 version of these configs caused ugly warnings to fly out of the dev server every time you started it. These are all cleanups recommended by the Tailwind 2->3 upgrade guide.

@vercel
Copy link

vercel bot commented Apr 19, 2022

@mqp is attempting to deploy a commit to the Manifold Markets Team on Vercel.

To accomplish this, @mqp needs to request access to the Team.

Afterwards, an owner of the Team is required to accept their membership request.

If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account.

Copy link
Member

@akrolsmir akrolsmir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (pending it works in Vercel). Thanks for the cleanup, those messages had been bothering me too!

@jahooma jahooma merged commit bf408d9 into manifoldmarkets:main Apr 20, 2022
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

3 participants