Skip to content

chore: fix script name and update CONTRIBUTING.md#325

Merged
hardfist merged 2 commits intoweb-infra-dev:mainfrom
Thiry1:fix-contributing-md
Sep 4, 2025
Merged

chore: fix script name and update CONTRIBUTING.md#325
hardfist merged 2 commits intoweb-infra-dev:mainfrom
Thiry1:fix-contributing-md

Conversation

@Thiry1
Copy link
Copy Markdown
Contributor

@Thiry1 Thiry1 commented Sep 3, 2025

Currently, building the project by following the instructions in CONTRIBUTING.md results in the following error.
I have added a step to apply a patch to typescript-go in CONTRIBUTING.md as part of the build process.

packages/rslint-api build$ rslib build
│   Rslib v0.12.4
│ info    build started...
│ ready   built in 0.13 s
│ start   generating declaration files... (esm)
│ info    declaration files prepared in 0.57 s (esm)
│ error   [api-extractor] Internal Error: Unable to determine semantic information for declaration:
│ /Users/***/workspace/rslint/typescript-go/_packages/api/src/client.ts:32:24
│ You have encountered a software defect. Please consider reporting the issue to the maintainers of this application. (esm)
│ File (esm)       Size      
│ dist/index.mjs   78.8 kB
│ error   Failed to build.
│ error   Error occurred in esm declaration files generation.
└─ Failed in 2.1s at /Users/***/workspace/rslint/packages/rslint-api

@netlify
Copy link
Copy Markdown

netlify bot commented Sep 3, 2025

Deploy Preview for rslint ready!

Name Link
🔨 Latest commit 98fef6c
🔍 Latest deploy log https://app.netlify.com/projects/rslint/deploys/68b7c4df6acb390008b53be3
😎 Deploy Preview https://deploy-preview-325--rslint.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Thiry1 Thiry1 changed the title chore: fix script name and update CONTRIBUTING.mdFix contributing md chore: fix script name and update CONTRIBUTING.md Sep 3, 2025
@hardfist
Copy link
Copy Markdown
Contributor

hardfist commented Sep 4, 2025

thanks!

@hardfist hardfist merged commit 4f7abab into web-infra-dev:main Sep 4, 2025
14 checks passed
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.

2 participants