Skip to content
This repository was archived by the owner on Nov 20, 2024. It is now read-only.

use_named_constants constant pattern tests #4211

Merged
merged 1 commit into from
Mar 24, 2023
Merged

use_named_constants constant pattern tests #4211

merged 1 commit into from
Mar 24, 2023

Conversation

pq
Copy link
Contributor

@pq pq commented Mar 23, 2023

srawlins
srawlins approved these changes Mar 23, 2023
View reviewed changes
Copy link
Contributor

@srawlins srawlins left a comment

Choose a reason for hiding this comment

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

🎉 I'm so happy this was already working, haha.

Thanks!

@pq pq merged commit 2a8f37b into main Mar 24, 2023
@pq pq deleted the use_named_constants_patterns branch March 24, 2023 00:01
@coveralls
Copy link

Coverage Status

Coverage: 96.329%. Remained the same when pulling 46683b3 on use_named_constants_patterns into 40dd80c on main.

copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request Aug 23, 2023
@pq
use_named_constants constant pattern tests (dart-archive/linter#4211)
40477c9
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@srawlins srawlins srawlins approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

use_named_constants should have tests with constant patterns
3 participants
@pq @coveralls @srawlins