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

chore: use set-github-repository-labels and populate-all-contributors-for-repository internally #1772

Merged
merged 9 commits into from
Dec 16, 2024

Conversation

JoshuaKGoldberg
Copy link
Owner

@JoshuaKGoldberg JoshuaKGoldberg commented Dec 16, 2024

PR Checklist

Overview

Brings on a few utilities I recently extracted:

These should all have the same functionality as before. This should be a chore: PR, not a feat: or fix:.

The utility here is that the latter two are now used as standalone scripts in onboarding to the "create" engine. I thought they'd be done as network requests (#1752), but in this way I found it a lot simpler to have them as standalone scripts (#1751, JoshuaKGoldberg/create#60).

💖

@@ -68,10 +70,11 @@ export const outcomeLabels = [
},
{
color: "d73a4a",
description: "Something isn't working :( 🐛",
description: "Something isn't working 🐛",
Copy link
Owner Author

Choose a reason for hiding this comment

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

Technically this is a behavior change in the repo. Heh. I'm curious if anybody will notice. The combo of an emoticon and an emoji bugs me.

@@ -214,6 +214,7 @@ exports[`expected file changes > knip.json 1`] = `
{
"$schema": "https://unpkg.com/knip@latest/schema.json",
- "entry": ["script/*e2e.js", "src/index.ts!", "src/**/*.test.*"],
- "ignoreDependencies": ["all-contributors-cli"],
Copy link
Owner Author

Choose a reason for hiding this comment

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

I think it was coming in as a nested dev dependency before. Now that I've had to make it explicit in package.json, Knip is reporting it. Ah well.

@JoshuaKGoldberg JoshuaKGoldberg force-pushed the set-github-repository-labels branch from e108fae to cb49cdf Compare December 16, 2024 18:55
@JoshuaKGoldberg JoshuaKGoldberg merged commit 340b8fb into main Dec 16, 2024
14 checks passed
@JoshuaKGoldberg JoshuaKGoldberg deleted the set-github-repository-labels branch December 16, 2024 19:08
Copy link

🎉 This is included in version v1.77.1 🎉

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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🛠 Tooling: Extract repository label setting into its own package
1 participant