ocsigen / ocsigen/tyxml

Naming consistency

Open
#164 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

breaking enhancement
Dominant language
OCaml
Stars
179
Forks
63
Avg merge
5h
Merged PRs (30d)
22

Description

I came across a possible naming inconsistency for `Class -- it seems to use `Style_Attr rather than the more consistent (to my mind) `Style_attr

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

The issue identifies Class and Style_Attr as the relevant names; start by locating their definitions and usages in the repository and compare them with Style_attr. Confirm the intended public naming, then update the affected references and run the repository's relevant checks so the names are consistent.

Written by the indexing model from the issue text.

Assessment

Tech stack
ocaml
Domain
developer-experience
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.