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

docs: add unique dependencies rule #620

Conversation

davidlj95
Copy link
Contributor

PR Checklist

Overview

Adds docs for unique-dependencies rule

Favourite emoji: 🥘

Copy link
Owner

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Choose a reason for hiding this comment

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

Looks great to me, thanks for the docs boost too!

Just requesting changes on formatting 🙂.

And the 🥘 is appreciated!!

docs/rules/unique-dependencies.md Show resolved Hide resolved
@JoshuaKGoldberg JoshuaKGoldberg added the status: waiting for author Needs an action taken by the original poster label Nov 12, 2024
@davidlj95 davidlj95 force-pushed the update-unique-dependencies-docs branch from ea0c5a8 to 1e41854 Compare November 12, 2024 20:02
@davidlj95
Copy link
Contributor Author

Just seen the eslint CI step failing. Taking a look... 👁️

@davidlj95 davidlj95 force-pushed the update-unique-dependencies-docs branch from 1e41854 to 301e2e4 Compare November 12, 2024 20:07
@davidlj95
Copy link
Contributor Author

Done! Just noticed that README.md is generated based on the rule description. And hadn't updated that :S Fixed 🔧

@davidlj95 davidlj95 force-pushed the update-unique-dependencies-docs branch from 301e2e4 to 30d4a7f Compare November 12, 2024 20:12
@davidlj95
Copy link
Contributor Author

And now saw the formatter was failing too 🙃 Just fixed it. Sorry for the spam 😅

Copy link
Owner

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Choose a reason for hiding this comment

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

Melissa Peterman as Bonnie Wheeler happily exclaiming "It's glorious!" with her hands in the air

@JoshuaKGoldberg JoshuaKGoldberg merged commit 42313ed into JoshuaKGoldberg:main Nov 12, 2024
12 checks passed
@JoshuaKGoldberg
Copy link
Owner

@all-contributors please add @davidlj95 for doc.

🤖 Beep boop! This comment was added automatically by all-contributors-auto-action.
Not all contributions can be detected from Git & GitHub alone. Please comment any missing contribution types this bot missed.
...and of course, thank you for contributing! 💙

Copy link
Contributor

@JoshuaKGoldberg

I've put up a pull request to add @davidlj95! 🎉

@davidlj95 davidlj95 deleted the update-unique-dependencies-docs branch November 12, 2024 20:20
JoshuaKGoldberg added a commit that referenced this pull request Nov 12, 2024
Adds @davidlj95 as a contributor for doc.

This was requested by JoshuaKGoldberg [in this
comment](#620 (comment))

---------

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
Co-authored-by: Josh Goldberg <git@joshuakgoldberg.com>
Copy link

🎉 This is included in version v0.16.0 🎉

The release is available on:

Cheers! 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: waiting for author Needs an action taken by the original poster
Projects
None yet
2 participants