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

Enable Travis-CI#8

Merged
kevmoo merged 3 commits into
masterfrom
travis
Oct 13, 2017
Merged

Enable Travis-CI#8
kevmoo merged 3 commits into
masterfrom
travis

Conversation

@kevmoo

@kevmoo kevmoo commented Oct 13, 2017

Copy link
Copy Markdown
Contributor

No description provided.

@kevmoo kevmoo requested a review from nex3 October 13, 2017 05:37
Comment thread .travis.yml Outdated
dart_task:
- test
- dartfmt
- dartanalyzer

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We shouldn't run the formatter and analyzer on both dev and stable; see the project template.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ack

Comment thread .travis.yml
@@ -0,0 +1,18 @@
language: dart

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Add sudo: false.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@kevmoo

kevmoo commented Oct 13, 2017

Copy link
Copy Markdown
Contributor Author

PTAL

@kevmoo kevmoo merged commit ce914cd into master Oct 13, 2017
@kevmoo kevmoo deleted the travis branch October 13, 2017 20:48
mosuem pushed a commit to dart-lang/test that referenced this pull request Dec 12, 2024
* dartfmt
* Rename analysis_options, enable travis
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants