Skip to content

feat!: add require-repository rule#1491

Merged
michaelfaith merged 4 commits intomichaelfaith:mainfrom
marcalexiei:require-repository
Jan 3, 2026
Merged

feat!: add require-repository rule#1491
michaelfaith merged 4 commits intomichaelfaith:mainfrom
marcalexiei:require-repository

Conversation

@marcalexiei
Copy link
Contributor

PR Checklist

Overview

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

@codecov
Copy link

codecov bot commented Jan 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.39%. Comparing base (f4f9430) to head (79de3e0).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1491   +/-   ##
=======================================
  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.

Beautiful, thanks for another great contribution! Just one request, and it should be good to go.

@michaelfaith michaelfaith changed the title feat: add require-repository rule feat!: add require-repository rule Jan 3, 2026
Repository owner deleted a comment from github-actions bot Jan 3, 2026
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.

Looking good, just thinking we might want to set ignorePrivate to true for this one.

@michaelfaith
Copy link
Owner

Updated the title to indicate it's a breaking change

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.

Great work. Thanks again for the contributions! 🚀

@michaelfaith michaelfaith merged commit 4d802c5 into michaelfaith:main Jan 3, 2026
15 checks passed
michaelfaith pushed a commit that referenced this pull request Jan 4, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.88.0](v0.87.1...v0.88.0)
(2026-01-04)


### ⚠ BREAKING CHANGES

* add require-repository rule
([#1491](#1491))

### Features

* add require-repository rule
([#1491](#1491))
([4d802c5](4d802c5))


### Bug Fixes

* **repository-shorthand:** support repo provider prefix
([#1472](#1472))
([fb5f10a](fb5f10a))
* **valid-repository:** support dashes and dots in repo shorthand
([#1468](#1468))
([8b83678](8b83678))

---
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>
@marcalexiei marcalexiei deleted the require-repository branch January 6, 2026 12:12
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-repository rule

2 participants