Robyn

Results 218 comments of Robyn

## Initial results from first prettier run Code style issues found in **437** files Click to see the list of files formatted ```plaintext .github/FUNDING.yml .github/ISSUE_TEMPLATE/bug_report.md .github/ISSUE_TEMPLATE/feature_request.md .github/workflows/test.js.yml CONTRIBUTING.md docs/_includes/download.html docs/_includes/header.html...

@vchendrix if ESS-Dive has requirements other than those listed here, it would be great to know about those!

### Initial ESlint report 1871 problems (1871 errors, 0 warnings), over 228 files [📄 eslint-errors.html.txt](https://github.com/NCEAS/metacatui/files/15311613/eslint-errors.html.txt) Note, these results are from a run in the `src` dir only, excluding `src/components`. We...

I've been experimenting with using a publicly available config of linting rules, [`airbnb-base`](https://www.npmjs.com/package/eslint-config-airbnb-base), which seem to be a popular set of standards to adopt. However, this leads to `30470` errors...

thanks @yvonnesjy! 🧠⚡ That's exactly the strategy we landed on in a discussion with the broader dev team. So the rules will be more strict, and will include the airbnb...

@yvonnesjy @ianguerin @ianguerin @rushirajnenuji - this PR is ready for review! To test the new formatting & linting tools in this PR: 1. `gh pr checkout 2412` 2. `npm install`...

Thanks for going through this @yvonnesjy! I recently discovered GitHub provides a UI for viewing PR changes commit by commit. The second commit is the one that changes nearly every...

@yvonnesjy thank you so much for reviewing so thoroughly! > typos in existing code might cause the formatter to misinterpret... if we ever run into surprise bugs that require rollbacks,...

MetacatUI currently uses RequireJS for dependency management. As part of our effort to benefit from newer JavaScript features and improve performance, we want to migrate to ES6 modules. The task...