Skip to content

[151] feat: include the current environment in a created number's friendlyName

This is a fairly simple change, and includes additional test coverage. I also included an explanatory comment, because it seemed a little bit long to work into the name of the new test case. However, I know there's a preference for that kind of documentation to be done via tests, so I'm glad to move it if reviewers prefer.

The contribution guidelines specify that I should run the linter before opening a MR. However, there's no lint command specified in package.json, and I don't see any eslint config files anywhere either. Running npx eslint as it is reveals a good number of fixable errors, including some in my changes here. I'll be happy to fix these by hand, or to put in a separate MR with the results of linting the entire project.

Closes #151 (closed)

Merge request reports

Loading