Skip to content

Conversation

@eksrha
Copy link
Member

@eksrha eksrha commented May 4, 2021

Additions

  • parsing simple json Files
  • parsing simple yaml Files

Removals

none

Changes

added subcommands parseJSON and parseYAML

Notes

Todos

Checklist

  • PR has an informative and human-readable title
  • Changes are limited to a single goal (no scope creep)
  • Code can be automatically merged (no conflicts)
  • Passes all existing automated tests
  • Any change in functionality is tested
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged / future todos are captured in comments
  • Visually tested in supported browsers and devices (see checklist below 👇)
  • Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
  • Reviewers requested with the Reviewers tool ➡️

Accessibility

  • Keyboard friendly
  • Screen reader friendly

Other

  • No linting errors or warnings

@eksrha eksrha merged commit 1bdb0ed into main May 4, 2021
@eksrha eksrha deleted the feature/parsing-yaml-and-json branch May 4, 2021 17:11
eksrha added a commit that referenced this pull request Jan 17, 2022
eksrha added a commit that referenced this pull request Jan 20, 2022
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