Skip to content

CI Builds with GitHub Actions #34614

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

Merged
merged 1 commit into from
Oct 23, 2019
Merged

Conversation

ethomson
Copy link
Contributor

I wanted to see if there was any interest in adding GitHub Actions to the party for validating pull request builds and merges into master.

This should be almost the same configuration as what you have for Azure Pipelines and Travis, except with a lack of caching. (Artifact caching is coming 🔜 to GitHub Actions, but I was curious how well it performed even without that.)

/cc @RyanCavanaugh

@orta
Copy link
Contributor

orta commented Oct 23, 2019

Nice, we've got a few places using actions around our infra:

And an almost ready for publishing blog post on it all 👍

@RyanCavanaugh RyanCavanaugh merged commit eb08ee6 into microsoft:master Oct 23, 2019
@RyanCavanaugh
Copy link
Member

Let's see what happens!

- name: Remove existing TypeScript
run: |
npm uninstall typescript --no-save
npm uninstall tslint --no-save
Copy link
Member

Choose a reason for hiding this comment

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

I'm pretty sure we can remove this from our scripts, since we don't use tslint anymore, XD

@weswigham
Copy link
Member

weswigham commented Oct 23, 2019

BTW @ethomson what's the open source project concurrent job/resource limit look like for actions? (Far as I can tell it's just "free", and the pricing model doesn't seem to have concurrency limits) Part of the reason we use pipelines for a bunch is because we have a lot of extra free extra parallel jobs allotted to us, which helps us do intensive on-demand testing around release periods without getting backlogged.

mheiber pushed a commit to mheiber/TypeScript that referenced this pull request Oct 25, 2019
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