Skip to content
This repository has been archived by the owner on Sep 21, 2023. It is now read-only.

docs(style_guide): update standard library style guide #625

Merged
merged 2 commits into from
Apr 28, 2023

Conversation

kt3k
Copy link
Member

@kt3k kt3k commented Apr 26, 2023

This PR updates the naming convention of Deno Standard Library.

We don't mention the usage of UPPER_SNAKE_CASE in the style guide, but UPPER_SNAKE_CASE have been used in many places. That caused the discussion denoland/std#3331.

This change adds the description about the usage of UPPER_SNAKE_CASE in deno_std, and resolves the conflict between the style guide and the existing code.

closes denoland/std#3331

Copy link
Member

@bartlomieju bartlomieju left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for making this more formal 👍

@ry ry merged commit 1182366 into denoland:main Apr 28, 2023
@kt3k kt3k deleted the update-style-guide branch April 28, 2023 04:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Should we export UPPER_CASE_SNAKE_CASE items?
3 participants