Skip to content
This repository has been archived by the owner on Apr 28, 2022. It is now read-only.

Actions #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Actions #2

wants to merge 1 commit into from

Conversation

bryceml
Copy link
Owner

@bryceml bryceml commented Apr 22, 2020

What does this PR do?

tests github actions

What issues does this PR fix or reference?

Fixes:

Previous Behavior

Remove this section if not relevant

New Behavior

Remove this section if not relevant

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

  • Docs
  • Changelog
  • Tests written/updated

Commits signed with GPG?

Yes/No

Please review Salt's Contributing Guide for best practices.

See GitHub's page on GPG signing for more information about signing commits with GPG.

Copy link

@s0undt3ch s0undt3ch left a comment

Choose a reason for hiding this comment

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

We can improve the pre-commit step

on:
push:

jobs:

Choose a reason for hiding this comment

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

Copy link
Owner Author

Choose a reason for hiding this comment

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

Is this better?

Choose a reason for hiding this comment

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

Think so

@bryceml bryceml force-pushed the actions branch 5 times, most recently from 955273f to d0e2cf0 Compare April 22, 2020 18:51
on:
push:

jobs:

Choose a reason for hiding this comment

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

Think so

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.

2 participants