Skip to content

Conversation

@eksrha
Copy link
Member

@eksrha eksrha commented Apr 9, 2021

[Short description explaining the high-level reason for the pull request]

Additions

Removals

Changes

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 8847122 into main Apr 9, 2021
@eksrha eksrha deleted the bugfix/fix-env-infosMergeMessage branch April 9, 2021 08:58
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