Skip to content
This repository was archived by the owner on Jan 14, 2025. It is now read-only.

Update .gitignore to new dart_tool pub cache #45

Merged
merged 3 commits into from
Mar 12, 2018
Merged

Update .gitignore to new dart_tool pub cache #45

merged 3 commits into from
Mar 12, 2018

Conversation

pq
Copy link
Contributor

@pq pq commented Mar 10, 2018

@pq pq requested a review from kevmoo March 10, 2018 22:43
@kevmoo
Copy link
Contributor

kevmoo commented Mar 10, 2018

You may not want to delete .pub yet – unless you're sure folks aren't going to develop against this package in any non-latest -dev release...

@pq
Copy link
Contributor Author

pq commented Mar 11, 2018

I think we should be safe since the only folks working on this are working in the SDK and so are sure to be very bleeding edge. You OK living dangerously? ;)

@kevmoo
Copy link
Contributor

kevmoo commented Mar 12, 2018

We should update the SDK constraint to exclude stable then...

Or just do an add and keep .pub – we can do a big cleanup of this once v2 ships

@pq
Copy link
Contributor Author

pq commented Mar 12, 2018

KK. Added .pub back. PTAL.

@pq pq merged commit c629908 into master Mar 12, 2018
@kevmoo kevmoo deleted the dart_tool branch July 18, 2018 20:39
mosuem pushed a commit to dart-lang/tools that referenced this pull request Dec 9, 2024
…_tool

Update .gitignore to new `dart_tool` pub cache
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

3 participants