Skip to content

Fix README template file extensions: use ts/js instead of typescript/javascript#327

Merged
RetricSu merged 2 commits intodevelopfrom
copilot/fix-readme-typo-typescript
Dec 8, 2025
Merged

Fix README template file extensions: use ts/js instead of typescript/javascript#327
RetricSu merged 2 commits intodevelopfrom
copilot/fix-readme-typo-typescript

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Dec 5, 2025

The README template was using {{LANGUAGE}} for file paths, resulting in invalid extensions like index.typescript instead of index.ts.

Changes

  • Added {{LANGUAGE_EXT}} template variable in src/templates/processor.ts that maps typescriptts, javascriptjs
  • Updated README template to use {{LANGUAGE_EXT}} for file paths while keeping {{LANGUAGE}} for prose (e.g., "write smart contracts in typescript")
  • Added package-lock.json to .gitignore (project uses pnpm)

Example

Before:

│   └── index.{{LANGUAGE}} # Contract implementation

After:

│   └── index.{{LANGUAGE_EXT}} # Contract implementation

Generated output now correctly shows index.ts/index.js instead of index.typescript/index.javascript.

Original prompt

This section details on the original issue you should resolve

<issue_title>v4 template readme typo</issue_title>
<issue_description>https://github.com/ckb-devrel/offckb/blob/develop/templates/v4/base-template/README.md.template#L87

launge is not working for typescript here, should be ts instead of typescript in file extension</issue_description>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

… full language names

Co-authored-by: RetricSu <23436060+RetricSu@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix typo in v4 template README for file extension Fix README template file extensions: use ts/js instead of typescript/javascript Dec 5, 2025
Copilot AI requested a review from RetricSu December 5, 2025 06:43
@RetricSu RetricSu marked this pull request as ready for review December 5, 2025 08:02
@RetricSu RetricSu merged commit ed91c20 into develop Dec 8, 2025
2 checks passed
@RetricSu RetricSu deleted the copilot/fix-readme-typo-typescript branch December 8, 2025 00:04
RetricSu added a commit that referenced this pull request Dec 8, 2025
* Add -y flag to skip confirmation prompt in deploy command (#326)

* Initial plan

* Add -y/--yes flag to deploy command to skip confirmation

Co-authored-by: RetricSu <23436060+RetricSu@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: RetricSu <23436060+RetricSu@users.noreply.github.com>

* Fix README template file extensions: use ts/js instead of typescript/javascript (#327)

* Initial plan

* Fix README template to use correct file extensions (ts/js) instead of full language names

Co-authored-by: RetricSu <23436060+RetricSu@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: RetricSu <23436060+RetricSu@users.noreply.github.com>

---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: RetricSu <23436060+RetricSu@users.noreply.github.com>
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.

v4 template readme typo

2 participants