Skip to content

Zig: Use standard char token#3264

Merged
RunDevelopment merged 1 commit intoPrismJS:masterfrom
RunDevelopment:issue3170-zig
Dec 9, 2021
Merged

Zig: Use standard char token#3264
RunDevelopment merged 1 commit intoPrismJS:masterfrom
RunDevelopment:issue3170-zig

Conversation

@RunDevelopment
Copy link
Copy Markdown
Member

Changes:

  • Added char token
  • Renamed builtin-types -> builtin-type

@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 2, 2021

JS File Size Changes (gzipped)

A total of 1 files have changed, with a combined diff of +3 B (+0.3%).

Details
file master pull size diff % diff
components/prism-zig.min.js 1.17 KB 1.17 KB +3 B +0.3%

Generated by 🚫 dangerJS against 22fc701

Copy link
Copy Markdown
Contributor

@hoonweiting hoonweiting left a comment

Choose a reason for hiding this comment

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

Looking at the keyword reference for the latest stable version (0.8.1), I think there are three new keywords: anyframe, anytype, and nosuspend. Since this PR is focused on using the char token and does not touch the keyword token at all, should adding these keywords be part of another PR instead? If so, then I think this PR is ready to go!

@RunDevelopment
Copy link
Copy Markdown
Member Author

I think there are three new keywords: anyframe, anytype, and nosuspend

Good catch! I made a separate PR for that (#3279).

@RunDevelopment RunDevelopment merged commit c3f9fb7 into PrismJS:master Dec 9, 2021
@RunDevelopment RunDevelopment deleted the issue3170-zig branch December 9, 2021 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants