dart-lang / pub-dev

The pub.dev website
https://pub.dev
BSD 3-Clause "New" or "Revised" License
777 stars 145 forks source link

Fx pkg_admin_page_test.dart for forks #7750

Closed jsoref closed 4 months ago

jsoref commented 4 months ago

My forks of this repository had to have a different name than dart-lang/pub-dev, so pkg/pub_integration/test/pkg_admin_page_test.dart would fail.

I don't think that the admin page test needs to assert that the repository has that name, so this changes the code to validate that the repository matches the github environment variable's declared repository (including owner) which seems reasonable.


Contribution guidelines:
- See our [contributor guide](https://github.com/dart-lang/.github/blob/main/CONTRIBUTING.md) for general expectations for PRs. - Larger or significant changes should be discussed in an issue before creating a PR. - Contributions to our repos should follow the [Dart style guide](https://dart.dev/guides/language/effective-dart) and use `dart format`. - Most changes should add an entry to the changelog and may need to [rev the pubspec package version](https://github.com/dart-lang/sdk/wiki/External-Package-Maintenance#making-a-change). - Changes to packages require [corresponding tests](https://github.com/dart-lang/.github/blob/main/CONTRIBUTING.md#Testing). Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.