Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

blast_repo fixes #3407

Merged
merged 2 commits into from Nov 18, 2022
Merged

blast_repo fixes #3407

merged 2 commits into from Nov 18, 2022

Conversation

devoncarew
Copy link
Member

GitHub Action
No Response

@devoncarew
blast_repo fixes
88b8a9c 
GitHub Action
No Response
@devoncarew
Copy link
Member Author

When run on the dev channel:

Analyzing ....

   info - lib/src/builder/builder.dart:57:31 - Avoid returning this from methods just to enable a fluent interface. - avoid_returning_this

@jakemac53
Copy link
Contributor

I can fix the new lint issue separately then land this

@devoncarew devoncarew merged commit f02a133 into master Nov 18, 2022
@devoncarew devoncarew deleted the blast_repo-2022_11_17T21_44_56 branch November 18, 2022 16:27
simolus3 pushed a commit to simolus3/build that referenced this pull request Dec 11, 2022
@devoncarew @simolus3
blast_repo fixes (dart-lang#3407)
aadd081 
GitHub Action
No Response
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jakemac53 jakemac53 jakemac53 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@devoncarew @jakemac53