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

set CI=true when launch process in actions runner. #374

Merged
merged 1 commit into from Mar 17, 2020

Conversation

TingluoHuang
Copy link
Member

@TingluoHuang TingluoHuang merged commit a5eb8cb into master Mar 17, 2020
@TingluoHuang TingluoHuang deleted the users/tihuang/cienv branch March 17, 2020 23:58
@thorn0
Copy link

thorn0 commented Apr 8, 2020

Would be good if you announced such changes somehow, ideally directly in GitHub UI. Yesterday this broke our CI. As it turned out, our tests contained code that depended on this variable. It took quite a while until I found out the reason of the failures. See prettier/prettier#7980

@TingluoHuang
Copy link
Member Author

@chrispat for an idea on where could we announce this kind of change. :)

@chrispat
Copy link
Member

chrispat commented Apr 9, 2020

There should have been a changelog entry as this went out. I will look and see if it didn't publish for some reason. I apologize for the inconvenience.

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.

None yet

4 participants