Skip to content
This repository has been archived by the owner on Jul 5, 2023. It is now read-only.

(feature) Updates starter workflow to use new yaml format for GitHub Actions v2 #942

Closed
wants to merge 1 commit into from

Conversation

booyaa
Copy link

@booyaa booyaa commented Sep 28, 2019

To address this issue: hashicorp/terraform-github-actions#67

Still need to update individual action references, but will do as separate PRs.

@booyaa booyaa requested a review from a team as a code owner September 28, 2019 10:21
@hashicorp-cla
Copy link

hashicorp-cla commented Sep 28, 2019

CLA assistant check
All committers have signed the CLA.

- name: filter-to-pr-open-synced
uses: actions/bin/filter@master
with:
args: action 'opened|synchronize'

Choose a reason for hiding this comment

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

This step is outdated in the YAML syntax. We've added event filtering directly to the YAML. You can achieve this by movig the types up into the on trigger above. In this case:

on:
  pull_request:
    types: [ opened, synchronize ]

However, the default triggers for on: pull_request are actually opened, synchronized, reopened which seem like reasonable defaults here, too, so you might just remove this step and not update the on: pull_request line at all.

Choose a reason for hiding this comment

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

Additionally with the latest release of 0.4.2, you no longer have to run these checks exclusively on PRs. You can just run it on a normal push

@nfagerlund
Copy link
Member

@booyaa @ethomson @James-Quigley I think this is superseded by #988, right? Shall I close?

@nfagerlund nfagerlund closed this Nov 14, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants