Skip to content

️Use ResourcePosition in more SyntaxValue paths - #561

Merged
toddjonker merged 1 commit into
ion-fusion:mainfrom
toddjonker:push-uurnwlyxkvwu
Sep 1, 2026
Merged

️Use ResourcePosition in more SyntaxValue paths#561
toddjonker merged 1 commit into
ion-fusion:mainfrom
toddjonker:push-uurnwlyxkvwu

Conversation

@toddjonker

Copy link
Copy Markdown
Member

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@toddjonker

Copy link
Copy Markdown
Member Author

Sorry for the push flailing, I'm having some difficulties with Jujutsu

@SharkBaitDLS

Copy link
Copy Markdown
Contributor

Have you looked into GitHub's new support for stacked PRs? It's much more in line with the JJ development flow and might help with this sort of series of changes.

@toddjonker

Copy link
Copy Markdown
Member Author

Have you looked into GitHub's new support for stacked PRs? It's much more in line with the JJ development flow and might help with this sort of series of changes.

I haven't, but I'll check it out this week.

Have you tried JJ? I'm quite liking it, and the Jujutsu VCS integration IDEA plugin is pretty great (and a former Amazonian, it seems).

I still have some issues rebasing though, I kinda miss having straightforward access to stable commits sometimes.

@toddjonker
toddjonker merged commit beeb8c0 into ion-fusion:main Sep 1, 2026
4 checks passed
@toddjonker
toddjonker deleted the push-uurnwlyxkvwu branch September 1, 2026 00:21
@SharkBaitDLS

Copy link
Copy Markdown
Contributor

I haven't tried it yet, I'm reasonably happy still with pure Git at work now that stacked PRs let me work in a more Amazon-y way with Git instead of every single branch only being allowed to be one commit ahead of main. I can definitely see the appeal but I like manually managing my commit and branch states still.

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