Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Long Layer Names and Dep Bumps #230

Merged
merged 6 commits into from
Mar 6, 2024
Merged

Long Layer Names and Dep Bumps #230

merged 6 commits into from
Mar 6, 2024

Conversation

stdavis
Copy link
Member

@stdavis stdavis commented Mar 5, 2024

No description provided.

dependabot bot and others added 6 commits March 5, 2024 23:08
Bumps the major-dependencies group with 5 updates:

| Package | From | To |
| --- | --- | --- |
| [@mdx-js/react](https://github.com/mdx-js/mdx/tree/HEAD/packages/react) | `2.3.0` | `3.0.1` |
| [jsdom](https://github.com/jsdom/jsdom) | `22.1.0` | `24.0.0` |
| [read-pkg](https://github.com/sindresorhus/read-pkg) | `8.1.0` | `9.0.1` |
| [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) | `4.5.2` | `5.1.5` |
| [vitest](https://github.com/vitest-dev/vitest/tree/HEAD/packages/vitest) | `0.34.6` | `1.3.1` |


Updates `@mdx-js/react` from 2.3.0 to 3.0.1
- [Release notes](https://github.com/mdx-js/mdx/releases)
- [Changelog](https://github.com/mdx-js/mdx/blob/main/changelog.md)
- [Commits](https://github.com/mdx-js/mdx/commits/3.0.1/packages/react)

Updates `jsdom` from 22.1.0 to 24.0.0
- [Release notes](https://github.com/jsdom/jsdom/releases)
- [Changelog](https://github.com/jsdom/jsdom/blob/main/Changelog.md)
- [Commits](jsdom/jsdom@22.1.0...24.0.0)

Updates `read-pkg` from 8.1.0 to 9.0.1
- [Release notes](https://github.com/sindresorhus/read-pkg/releases)
- [Commits](sindresorhus/read-pkg@v8.1.0...v9.0.1)

Updates `vite` from 4.5.2 to 5.1.5
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v5.1.5/packages/vite)

Updates `vitest` from 0.34.6 to 1.3.1
- [Release notes](https://github.com/vitest-dev/vitest/releases)
- [Commits](https://github.com/vitest-dev/vitest/commits/v1.3.1/packages/vitest)

---
updated-dependencies:
- dependency-name: "@mdx-js/react"
  dependency-type: direct:development
  update-type: version-update:semver-major
  dependency-group: major-dependencies
- dependency-name: jsdom
  dependency-type: direct:development
  update-type: version-update:semver-major
  dependency-group: major-dependencies
- dependency-name: read-pkg
  dependency-type: direct:development
  update-type: version-update:semver-major
  dependency-group: major-dependencies
- dependency-name: vite
  dependency-type: direct:development
  update-type: version-update:semver-major
  dependency-group: major-dependencies
- dependency-name: vitest
  dependency-type: direct:development
  update-type: version-update:semver-major
  dependency-group: major-dependencies
...

Signed-off-by: dependabot[bot] <[email protected]>
I had some test users complain about this in enviro.

Ref: agrc/deq-enviro#651
Copy link
Member

@steveoh steveoh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this looks pretty normal although I don't know how it affects really long layer names. not having a max width seems like it could cause other problems.

@stdavis stdavis merged commit ab829e0 into main Mar 6, 2024
3 checks passed
@stdavis stdavis deleted the fix/long-layer-names branch March 6, 2024 00:18
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