Skip to content

Conversation

@helin24
Copy link
Member

@helin24 helin24 commented Jan 23, 2026

This deletes the product-matrix.json file and places that use it. I checked that bin/plugin generate still works (though we probably don't need this either) to make sure the plugin tool still compiles.

Related to #8723

@helin24 helin24 requested review from jwren and pq January 23, 2026 00:02
Copy link
Collaborator

@pq pq left a comment

Choose a reason for hiding this comment

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

Awesome!

.toList();
}

bool isTravisFileValid() {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Since this isn't really about travis anymore, consider changing to something like isPresubmitFileValid?

}

bool isTravisFileValid() {
var travisPath = p.join(rootPath, '.github/workflows/presubmit.yaml');
Copy link
Collaborator

Choose a reason for hiding this comment

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

Rename to presubmitPath?


bool isTravisFileValid() {
var travisPath = p.join(rootPath, '.github/workflows/presubmit.yaml');
var travisFile = File(travisPath);
Copy link
Collaborator

Choose a reason for hiding this comment

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

And presubmitFile?

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