Skip to content

Comments

fix: haskell to match style guide#63

Open
jack-mil wants to merge 1 commit intomainfrom
fix/haskell
Open

fix: haskell to match style guide#63
jack-mil wants to merge 1 commit intomainfrom
fix/haskell

Conversation

@jack-mil
Copy link
Member

The style CAPITOL seems to apply to all identifiers starting with a capitol letter. Does this have semantic meaning in Haskell? What is a reasonable color for it? red? text?

There are some styles that I cannot identify. Namely DATA, IMPORT, and INSTANCE. I tried several Haskell syntax examples and could not get these styles to trigger. If someone more familiar with Haskell can show an example of what it might be, please do. Maybe one day someone will come across it and report.

image

The style CAPITOL seems to apply to all identifiers starting with a
capitol letter. Does this have semantic meaning in Haskell? What is a
reasonable color for it? `red`? `text`?

There are some styles that I cannot identify. Namely DATA, IMPORT, and
INSTANCE. I tried several Haskell syntax examples and could not get
these styles to trigger. If someone more familiar with Haskell can show
an example of what it might be, please do. Maybe one day someone will
come across it and report.
@jack-mil jack-mil changed the title fix: haskell to match match style guide fix: haskell to match style guide Apr 29, 2025
@typedrat
Copy link

typedrat commented Apr 29, 2025

  1. Capital letters versus lowercase letters at the start of an identifier in Haskell separate the variable and constructor namespaces (see varId and conId in the Haskell 2010 Report, section 10.2 on lexical syntax).
  2. I'll have to read the actual definition of the syntax for Notepad++, but I'd assume that DATA, IMPORT, and INSTANCE not triggering is a bug, since those are major families of declaration statement (see topdecl in the Haskell 2010 Report, section 10.5 on context-free syntax).

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.

2 participants