Skip to content

Commit

Permalink
Update open questions in design spec
Browse files Browse the repository at this point in the history
  • Loading branch information
pniederw committed Oct 7, 2014
1 parent 60a27b7 commit 9a12043
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions design-docs/sharing-code-quality-config.md
Original file line number Diff line number Diff line change
Expand Up @@ -155,6 +155,12 @@ This can be improved later.
- Should `TextResource` methods throw `IOException` - (I don't think there's a point - LD)
- Should `TextResource.as*()` throw if called when the `getTaskDependencies()` are unsatisfied (e.g. can't call this at config time)
- For the to-be deprecated `File` properties (e.g. `Checkstyle.configFile`), do we support calling the getter if the `TextResource` property was set?
- It might be better to have separate methods rather than overloads (e.g. `resources.textFromString()` and `resources.textFromFile()`, or
`resources.text.fromString()` and `resources.text.fromFile()`). This way we could support the usual coercions (e.g. `String`->`File`),
although we'd still have to deal with the fact that the same method needs to accept both files and single-element file collections.
- It feels that the `resources.text()`/`resources.archiveText()` methods compensate for limitations in Gradle's file APIs. For example,
it would be more natural to select an archive entry using a file (tree) API and pass the result to `resources.text()`, rather than
using `resources.archiveText()`.



Expand Down

0 comments on commit 9a12043

Please sign in to comment.