Skip to content

properties: Use key, value for token names, attr-... as aliases#3377

Merged
RunDevelopment merged 1 commit intoPrismJS:masterfrom
hoonweiting:3170-standardise-properties
Mar 9, 2022
Merged

properties: Use key, value for token names, attr-... as aliases#3377
RunDevelopment merged 1 commit intoPrismJS:masterfrom
hoonweiting:3170-standardise-properties

Conversation

@hoonweiting
Copy link
Contributor

This PR replaces attr-name and attr-value with key and value, and uses attr-... as aliases instead.

Overall, this brings properties in line with similar languages (INI, editorconfig, systemd...).

@github-actions
Copy link

github-actions bot commented Mar 9, 2022

JS File Size Changes (gzipped)

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

Details
file master pull size diff % diff
components/prism-properties.min.js 190 B 203 B +13 B +6.8%

Generated by 🚫 dangerJS against 3be90a6

@RunDevelopment RunDevelopment merged commit b94a664 into PrismJS:master Mar 9, 2022
@RunDevelopment
Copy link
Member

Thank you for the PR @hoonweiting!

I think that was the last config-type language with that problem, right?

@hoonweiting
Copy link
Contributor Author

I'm not really sure to be honest! I was just looking at the list of languages that haven't been completed, so I might have missed out on some config-type languages.

I'm looking at another language right now, but I'll go through the list again another day to check if there are more config-type languages! :)

@hoonweiting hoonweiting deleted the 3170-standardise-properties branch March 9, 2022 18:11
@RunDevelopment
Copy link
Member

Thank you!

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