afids / afids-validator

Validator for the anatomical fiducial placement protocol
https://validator.afids.io
GNU General Public License v3.0
2 stars 4 forks source link

BUG: Update workflow #121

Closed kaitj closed 3 years ago

kaitj commented 3 years ago

Proposed changes

This resolves the actions bug with triggering on draft. Removed the additional conditionals ([skip ci] and [wip]) as it caused the actions to always trigger. [skip ci] is actually built into GH actions now, so we don't need to separately check for that tag. I've suggested in the contributing docs to either add [skip ci] to commit messages or to use a draft PR for WIP material.

One thing I did not include is the output of the payload information to debug actions. I think this something that should added to one's own fork for testing github actions rather than including here. We can however include it if others feel like we should. Not sure how it handles secrets stored in the environment (e.g. passwords, keys, for triggering heroku deployment).

Types of changes

What types of changes does your code introduce? Put an x in the boxes that apply

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you are unsure about any of the choices, don't hesitate to ask!

Notes

All PRs will undergo the unit testing before being reviewed. You may be requested to explain or make additional changes before the PR is accepted.

_PR template was adopted from appium_

kaitj commented 3 years ago

CI currently failing on black as it's not set up to check from a forked branch. This may be something to look into in the future, but I don't think it should prevent us from merging this in.