google / google/grinder.dart

Make `Grinder.tasks` a `Map`, remove `Grinder.getTask`

Open
#282 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Dart
Stars
266
Forks
36
Avg merge
23h 25m
Merged PRs (30d)
2

Description

Currently you can access the tasks via `grinder.tasks` or look up an individual task via `grinder.getTask(name)`. But those are disjoint from each other, and accessing just the task names requires something like `grinder.tasks.map((task) => task.name)`.

I think a more intuitive and idiomatic API would be:

`Map get tasks;`

It's a breaking change, but it's unlikely to break many if any folks yet.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.