Uh oh!
There was an error while loading. Please reload this page.
Expand coverage for "bundle summary" - #3130
Merged
Merged
Conversation
denik
requested review from
andrewnester, anton-107, pietern and shreyas-goenka
as code ownersJune 26, 2025 14:03
denikforce-pushed
the
denik/TEST_DEFAULT_WAREHOUSE_ID
branch
from
June 27, 2025 08:03
8fa8e63 to
e0405d9Comparedenikforce-pushed
the
denik/TEST_DEFAULT_WAREHOUSE_ID
branch
from
June 27, 2025 08:05
e0405d9 to
1a78eaaCompareeng-dev-ecosystem-bot
commented
Jun 27, 2025
Collaborator
shreyas-goenka
approved these changes
Jun 27, 2025
denikforce-pushed
the
denik/TEST_DEFAULT_WAREHOUSE_ID
branch
from
June 27, 2025 09:21
ae5bc47 to
1c5145cComparedenik added a commit
that referenced
this pull request
May 20, 2026
## Tests For jobs, dashboards, secret scopes check that "id" and "modified_status" fields are set correctly by printing more output from "bundle summary". This highlights the issue with secret scopes - modified_status is not correct there. ## Why To support refactoring (#3134) and adding support for direct deployment to bundle summary.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Tests
For jobs, dashboards, secret scopes check that "id" and "modified_status" fields are set correctly by printing more output from "bundle summary".
This highlights the issue with secret scopes - modified_status is not correct there.
Why
To support refactoring (#3134) and adding support for direct deployment to bundle summary.