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

Add please to the website #87

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add please to the website #87

wants to merge 3 commits into from

Conversation

Tatskaari
Copy link

@Tatskaari Tatskaari commented Dec 14, 2022

Fixes: #67

@netlify
Copy link

netlify bot commented Dec 14, 2022

Deploy Preview for monorepotools ready!

Name Link
🔨 Latest commit 87a35e7
🔍 Latest deploy log https://app.netlify.com/sites/monorepotools/deploys/639afd66bdc3150008ad627c
😎 Deploy Preview https://deploy-preview-87--monorepotools.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@juristr
Copy link
Member

juristr commented Dec 15, 2022

thanks for submitting. I'll have a look later today/tomorrow. I think the build failed due to format issues. Can you run yarn nx format locally and then push again?

@Tatskaari
Copy link
Author

thanks for submitting. I'll have a look later today/tomorrow. I think the build failed due to format issues. Can you run yarn nx format locally and then push again?

Thanks! I've gotten an internal review from our side so we're happy to land when you are.

@Tatskaari
Copy link
Author

Hey! Hope you had a good holiday break. Any chance of a review here?

@Drena2020
Copy link

Drena2020 commented Jan 3, 2023 via email

@bcabanes
Copy link
Member

Hi! Thank you for the PR, we are on a stretch right now but will attend to it. This one a delicate since the design needs to be adjusted to have more and more tools to show.

We'll come back to you!

@galuszkak
Copy link

Hey @bcabanes

Is there any progress on this?

@Tatskaari
Copy link
Author

Hey guys! Just a friendly bump to see if there's any progress here. :)

@ZackDeRose
Copy link
Member

ZackDeRose commented Sep 18, 2023

Hey @Tatskaari - I'm taking over for the monorepo.tools for now - I had a chance to look over please and I think it's a legitimate addition to the site.

I think your submission is mostly fine. Can you:

  1. rebase your PR with the latest
  2. adjust the order for please so that it fits in alphbetically with the others

I'll look forward to adding your submission when these are done.

@ZackDeRose
Copy link
Member

Hey @Tatskaari - just sending a friendly poke! Looks like this PR was left out a bit in the past, but I'm actively managing the repo from now on!

Let me know if you're still interested in including please!

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.

Include please build tool
6 participants