Skip to content

feat: add require-homepage rule#1480

Merged
michaelfaith merged 4 commits intomichaelfaith:mainfrom
marcalexiei:rule-require-homepage
Dec 31, 2025
Merged

feat: add require-homepage rule#1480
michaelfaith merged 4 commits intomichaelfaith:mainfrom
marcalexiei:rule-require-homepage

Conversation

@marcalexiei
Copy link
Contributor

PR Checklist

Overview

Add homepage to properties listed in src/rules/require-properties.ts

@codecov
Copy link

codecov bot commented Dec 31, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.39%. Comparing base (6d83a7f) to head (b2769b3).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1480   +/-   ##
=======================================
  Coverage   97.39%   97.39%           
=======================================
  Files          28       28           
  Lines         575      575           
  Branches      169      169           
=======================================
  Hits          560      560           
  Misses          1        1           
  Partials       14       14           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Owner

@michaelfaith michaelfaith left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution! 🚀 Just a couple small requests from me.

Copy link
Owner

@michaelfaith michaelfaith left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks again 👍

@michaelfaith michaelfaith merged commit 0d895ff into michaelfaith:main Dec 31, 2025
15 checks passed
@michaelfaith
Copy link
Owner

@all-contributors please add @marcalexiei for code

@allcontributors
Copy link
Contributor

@michaelfaith

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

michaelfaith pushed a commit that referenced this pull request Dec 31, 2025
Adds @marcalexiei as a contributor for code.

This was requested by michaelfaith [in this
comment](#1480 (comment))

---------

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
michaelfaith added a commit that referenced this pull request Jan 1, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.87.1](v0.87.0...v0.87.1)
(2025-12-31)


### Features

* add require-homepage rule
([#1480](#1480))
([0d895ff](0d895ff))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Josh Goldberg ✨ <git@joshuakgoldberg.com>
Co-authored-by: michael faith <michaelfaith@users.noreply.github.com>
michaelfaith pushed a commit that referenced this pull request Jan 1, 2026
<!-- 👋 Hi, thanks for sending a PR to eslint-plugin-package-json! 🗂
Please fill out all fields below and make sure each item is true and [x]
checked.
Otherwise we may not be able to review your PR. -->

## PR Checklist

- [ ] Addresses an existing open issue: fixes #000
- [ ] That issue was marked as [`status: accepting
prs`](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/issues?q=is%3Aopen+is%3Aissue+label%3A%22status%3A+accepting+prs%22)
- [x] Steps in
[CONTRIBUTING.md](https://github.com/JoshuaKGoldberg/eslint-plugin-package-json/blob/main/.github/CONTRIBUTING.md)
were taken

> [!NOTE]
> Actually this is a followup of #1480

## Overview

I noticed that were some typescript errors in 
- `eslint.config.ts` - `perfectionist.configs` is possibly undefined
- `vitest.config.ts` - `coverage.all` option no longer is not a valid
option in `vitest@4`

To fix them I added `*.config.ts` to `tsconfig#includes` and updated
`languageOptions.parserOptions.projectService` accordingly
@marcalexiei marcalexiei deleted the rule-require-homepage branch January 1, 2026 22:51
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.

🚀 Feature: Bring in a require-homepage rule

2 participants