Skip to content

Remove extra version info#2539

Merged
natebosch merged 9 commits intomasterfrom
drop-extra-version-info
Apr 13, 2026
Merged

Remove extra version info#2539
natebosch merged 9 commits intomasterfrom
drop-extra-version-info

Conversation

@natebosch
Copy link
Copy Markdown
Member

@natebosch natebosch commented Sep 6, 2025

Follow up to #2538

This is only relevant when using the test runner as a git or path
dependency. In these cases the version will already end in -WIP with
our current development practices and in the places where it may come up
the users don't need help understanding where the code is coming from.

Closes #2535

Add a check for the `workspace_ref.json` file in place of the
`pubspec.lock` file that exists for single package pub solutions.

When the workspace ref exists, follow it and read the
`package_graph.json` file from the workspace root.
The test had been passing because the `testVersion` variable was null in
the pub workspace which matched the behavior of the test_descriptor
package within the test..
Follow up to #2538

This is only relevant when using the test runner as a git or path
dependency. In these cases the version will already end in `-WIP` with
out current development practices and in the places where it may come up
the users don't need help understanding where the code is coming from.
@natebosch natebosch changed the title drop extra version info Remove extra version info Sep 6, 2025
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Sep 6, 2025

PR Health

@natebosch natebosch requested a review from kevmoo April 13, 2026 23:02
@natebosch natebosch marked this pull request as ready for review April 13, 2026 23:05
@natebosch natebosch requested a review from a team as a code owner April 13, 2026 23:05
Copy link
Copy Markdown
Member

@kevmoo kevmoo left a comment

Choose a reason for hiding this comment

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

Nice cleanup!

@natebosch natebosch merged commit bd40148 into master Apr 13, 2026
62 checks passed
@natebosch natebosch deleted the drop-extra-version-info branch April 13, 2026 23:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants