Skip to content

BREAKING CHANGE: use JSONPath indexes#268

Merged
irahopkinson merged 1 commit intomainfrom
usj-location-change
Mar 19, 2025
Merged

BREAKING CHANGE: use JSONPath indexes#268
irahopkinson merged 1 commit intomainfrom
usj-location-change

Conversation

@irahopkinson
Copy link
Copy Markdown
Collaborator

  • use JSONPath indexes instead of JSONPath for USJ location
  • add utilities to convert between JSONPath indexes and JSONPath (for backwards compatibility)
  • also fix package.json build warnings
  • also export more types

- use JSONPath indexes instead of JSONPath for USJ location
- add utilities to convert between JSONPath indexes and JSONPath (for backwards compatibility)
- also fix `package.json` build warnings
- also export more types
@codesandbox
Copy link
Copy Markdown

codesandbox bot commented Mar 19, 2025

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@nx-cloud
Copy link
Copy Markdown

nx-cloud bot commented Mar 19, 2025

View your CI Pipeline Execution ↗ for commit c83b0a8.

Command Status Duration Result
nx run-many -t build ✅ Succeeded 51s View ↗
nx run-many -t test ✅ Succeeded 53s View ↗
nx run-many -t typecheck ✅ Succeeded 21s View ↗
nx run-many -t lint ✅ Succeeded 5s View ↗

☁️ Nx Cloud last updated this comment at 2025-03-19 01:43:13 UTC

@irahopkinson irahopkinson merged commit d8ebcfc into main Mar 19, 2025
5 checks passed
@irahopkinson irahopkinson deleted the usj-location-change branch March 19, 2025 01:48
irahopkinson added a commit that referenced this pull request Mar 21, 2025
- This partially reverts commit d8ebcfc.
- `README.md`, `App.tsx`, & `selection.model.ts` were reverted,  other files modified to suit.
irahopkinson added a commit that referenced this pull request Mar 21, 2025
- This partially reverts commit d8ebcfc.
- `README.md`, `App.tsx`, & `selection.model.ts` were reverted,  other files modified to suit.
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.

1 participant