Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor build process from Python to JavaScript #78

Merged
merged 8 commits into from
Dec 5, 2024
Merged

Conversation

graphemecluster
Copy link
Member

  • Replace Python build script with JavaScript
    • The 選項 argument of from字頭 now accepts both objects and arrays. (Allowing specifying objects is purely for backward compatibility.) Results are outputted as both array elements and named properties. Closes from字頭選項 引數應為列表 #77.
  • Check coverage during building and testing (Fixes CI 測試的方案覆蓋檢查 #71)
  • Separate type definitions to the dedicated file such that they are recognised when called in a TypeScript file
  • Move generated artifacts to a dist directory and make the npm package include only this directory

- Check coverage during building & testing
- Generate & Bundle type definition file; Move JSDoc to there
- Update workflow
- `index.js` artifact is now located in `dist`.
- Files to ignore are now specified by CLI flag instead of inside config to get rid of `Unused eslint-disable directive` warning in `index.js`.
eslint.config.js Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
@syimyuzya
Copy link
Member

Also, that "Python" in the VSCode extension recommendations can be removed (and replaced by ESLint).

package.json Outdated Show resolved Hide resolved
@graphemecluster
Copy link
Member Author

@syimyuzya 你 commit 啦,我這幾天 AFK 😄

@syimyuzya
Copy link
Member

@syimyuzya 你 commit 啦,我這幾天 AFK 😄

好嗒~

@syimyuzya syimyuzya merged commit 4b4441d into main Dec 5, 2024
1 check 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.

from字頭選項 引數應為列表 CI 測試的方案覆蓋檢查
2 participants