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

CI: Don't to run the tests if only the docs were updated (WIP) #6999

Closed
wants to merge 11 commits into from
Closed

CI: Don't to run the tests if only the docs were updated (WIP) #6999

wants to merge 11 commits into from

Conversation

data-man
Copy link
Contributor

No description provided.

@data-man
Copy link
Contributor Author

Currently for the Travis only.

@dom96
Copy link
Contributor

dom96 commented Dec 30, 2017

Nice :o

@data-man
Copy link
Contributor Author

The grep not pre-installed on the AppVeyor :-(

@ghost
Copy link

ghost commented Jan 1, 2018

@data-man maybe you can do the same using PowerShell on AppVeyor?

@data-man
Copy link
Contributor Author

data-man commented Jan 1, 2018

@Yardanico
It turned out that grep is installed, just not mentioned in AppVeyor's tools.
I created repo for CI-experiments so the victory will be soon. :-)

@Araq
Copy link
Member

Araq commented Jan 1, 2018

The idea is bad. We check that the documentation builds too via koch web and typos in the RST files can break the build.

@Araq Araq closed this Jan 1, 2018
@data-man
Copy link
Contributor Author

data-man commented Jan 1, 2018

@Araq
But ./koch web executes!
And this PR in WIP.

@Araq Araq reopened this Jan 1, 2018
@data-man data-man closed this Mar 5, 2018
@ghost
Copy link

ghost commented Mar 5, 2018

@data-man why close?

@data-man
Copy link
Contributor Author

data-man commented Mar 6, 2018

@Yardanico
I don't have time to do this.

@timotheecour
Copy link
Member

attempted to revive it here #8552

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants