Skip to content

Added new build script (tsc) default one#2

Open
kisshan13 wants to merge 1 commit into
Muhammad-Owais-Warsi:masterfrom
kisshan13:new-build-system
Open

Added new build script (tsc) default one#2
kisshan13 wants to merge 1 commit into
Muhammad-Owais-Warsi:masterfrom
kisshan13:new-build-system

Conversation

@kisshan13

Copy link
Copy Markdown

Created a new build system and updated dev-dependencies.

To build use :

npm run build

New import maps added to import providers

Example :

import polar from "better-pay/providers/polar"

Current codebase contains few cyclic imports which needs to be remove and is part of #1

Comment thread package.json
"build": "tsup"
"dev": "tsc --watch",
"clean": "rimraf ./dist",
"build": "rimraf ./dist && tsc --incremental false"

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

maybe we can seperate the commands here

   "dev": "tsc --watch",
   "clean": "rimraf ./dist",
   "build": "rimraf ./dist && tsc --incremental false",
   "cache-build": "tsc --incremental"

@@ -1,4 +1,4 @@
import { baseProviderConfig, resultType } from "../../../types";
import { baseProviderConfig, resultType } from "../../types.js";

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

change formats back to ts

Comment thread package.json
"types": "./dist/index.d.ts",
"scripts": {
"build": "tsup"
"dev": "tsc --watch",

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

use tsup

Comment thread yarn.lock

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

we can use bun

@Muhammad-Owais-Warsi Muhammad-Owais-Warsi mentioned this pull request Aug 5, 2025
Closed
@Muhammad-Owais-Warsi Muhammad-Owais-Warsi added invalid This doesn't seem right question Further information is requested labels Sep 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

invalid This doesn't seem right question Further information is requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants