Skip to content

Add .editorconfig; fix some error messages in the parser#207

Merged
ToruNiina merged 2 commits intoToruNiina:masterfrom
cxw42:misc
Feb 11, 2023
Merged

Add .editorconfig; fix some error messages in the parser#207
ToruNiina merged 2 commits intoToruNiina:masterfrom
cxw42:misc

Conversation

@cxw42
Copy link
Contributor

@cxw42 cxw42 commented Jan 14, 2023

This is my first PR; thanks for considering it! It includes minor cleanup items.

  • Add an EditorConfig file to set up editors for what appear to be the standards of the codebase (four-space indents in C++, two-space indents in md/yaml)
  • Some error messages in the parser printed the wrong function names. Fix those.

cxw42 added 2 commits January 14, 2023 18:18
Four-space indents in C++ files; two-space indents in Markdown and YAML.
- fix error messages that referred to the wrong functions.
- parse_key(): remove "detail::" from the only error message that had
  it, for consistency with the other error messages in that function
@ToruNiina ToruNiina merged commit ff48387 into ToruNiina:master Feb 11, 2023
@cxw42 cxw42 deleted the misc branch April 10, 2023 01:45
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