Summary

Add TypeScript definitions for the build.

Review Request #14635 — Created Oct. 15, 2025 and discarded Oct. 25, 2025, 9:46 p.m.

Information

Owner

david

Repository

beanbag-eslint-plugin

Branch

master

Bugs

Depends On

Reviewers

Groups

beanbag-misc

People

Description

With the newer JS-based eslint configs, we import
@beanbag/eslint-plugin, resulting in warnings about it not having
definitions files. This change adds that, and switches up our build to
work more similarly to other JS packages we have (using rollup instead
of just invoking babel).

Testing Done

Did a build and saw that I no longer got warnings about missing
definitions.

Commits

Summary	ID
Add TypeScript definitions for the build. With the newer JS-based eslint configs, we import @beanbag/eslint-plugin, resulting in warnings about it not having definitions files. This change adds that, and switches up our build to work more similarly to other JS packages we have (using rollup instead of just invoking babel). Testing Done: Did a build and saw that I no longer got warnings about missing definitions.	ymzpvrzovplrzkrtoqmwvpmmxqpumqkt

Summary

Add TypeScript definitions for the build.

With the newer JS-based eslint configs, we import @beanbag/eslint-plugin, resulting in warnings about it not having definitions files. This change adds that, and switches up our build to work more similarly to other JS packages we have (using rollup instead of just invoking babel). Testing Done: Did a build and saw that I no longer got warnings about missing definitions.

ymzpvrzovplrzkrtoqmwvpmmxqpumqkt

flake8 passed.

JSHint passed.

Change Summary:

Bump dependencies too.

Commits:

	Summary	ID
	Add TypeScript definitions for the build. With the newer JS-based eslint configs, we import @beanbag/eslint-plugin, resulting in warnings about it not having definitions files. This change adds that, and switches up our build to work more similarly to other JS packages we have (using rollup instead of just invoking babel). Testing Done: Did a build and saw that I no longer got warnings about missing definitions.	ymzpvrzovplrzkrtoqmwvpmmxqpumqkt
	Add TypeScript definitions for the build. With the newer JS-based eslint configs, we import @beanbag/eslint-plugin, resulting in warnings about it not having definitions files. This change adds that, and switches up our build to work more similarly to other JS packages we have (using rollup instead of just invoking babel). Testing Done: Did a build and saw that I no longer got warnings about missing definitions.	ymzpvrzovplrzkrtoqmwvpmmxqpumqkt

Diff:

Revision 2 (+1620 -246)

Show changes

	.gitignore
	NEWS.md
	.babelrc
	package.json
	package-lock.json
	rollup.config.js
	tsconfig.json

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Status:: Discarded